Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

App-wide UI styling #76

Merged
merged 61 commits into from
Dec 13, 2024
Merged

App-wide UI styling #76

merged 61 commits into from
Dec 13, 2024

Conversation

jackcasstlesjones
Copy link
Collaborator

@jackcasstlesjones jackcasstlesjones commented Dec 12, 2024 •
edited
Loading

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update
  • Styling
  • Build
  • Chore

Description

App-wide UI styling, as discussed

Screenshots

Screenshot 2024-12-13 at 19 40 11
Screenshot 2024-12-13 at 19 40 20
Screenshot 2024-12-13 at 19 40 28
Screenshot 2024-12-13 at 19 40 40
Screenshot 2024-12-13 at 19 40 53

UI accessibility checklist

If your PR includes UI changes, please utilize this checklist:

  • Semantic HTML implemented?
  • Checked with axe DevTools and addressed Critical and Serious issues?
  • Color contrast tested?

Added/updated tests?

Please aim to keep the code coverage percentage at 80% and above.

  • Yes
  • No, and this is why: please replace this line with details on why tests
    have not been included
  • I need help with writing tests

Delete branch after merge?

  • Yes
  • No

What gif best describes this PR or how it makes you feel?

alt_text

@jackcasstlesjones
style: floating button now big and round"
eb5f4c6 
"
@jackcasstlesjones
chore: remove unused imports
f195458
@jackcasstlesjones
style: make add category button pretty
88a4e32
@jackcasstlesjones
style: add roboto font
f0572cc
@jackcasstlesjones
style: reduce size and boldness of subtitle
813657e
@jackcasstlesjones
feat: add hold and drag instruction
b398283
@jackcasstlesjones
style: add more definition to toolkit items
1d77ecb
@jackcasstlesjones
style: add more vertical whitespace to toolkit item"
ce42a86
@jackcasstlesjones
style: remove bottom border from categories bar
158e6c3
@jackcasstlesjones
style: change link colour
a3250d3
@jackcasstlesjones
style: change link colour
9577f1e
@jackcasstlesjones
style: add button styling to toolkit items delete
74df375
@jackcasstlesjones
chore: add aria-label to floatingbutton
e8997f9
@jackcasstlesjones
style: change back button in addtoolkit to chevron
2012942
@jackcasstlesjones
style: add placeholders to add tool and change borders
443b2fc
@jackcasstlesjones
style: remove focus outline none and increase white space in add tool…
e98b9c1 
… item
@jackcasstlesjones
style: spacing in add tool
370c626
@jackcasstlesjones
style: make add category button smaller
ea6192d
@jackcasstlesjones
style: make delete button use secondary twd purp
74a02e3
@jackcasstlesjones
style: increase vertical padding on delete button"
72fde36
@jackcasstlesjones
style: remove border from floating button
66a64c8
@jackcasstlesjones
style: make categories font lighter"
3d5e5ec
@jackcasstlesjones
style: change add category button to icon on addtool page
eaf67fd
@jackcasstlesjones
style: change category boldness in addtool
171ce89
@jackcasstlesjones
style: gradient on floating button
0b235eb
@jackcasstlesjones
style: add gradient to background and remove background from category…
452f6c0 
… bar
@netlify Netlify
Copy link

netlify bot commented Dec 12, 2024 •
edited
Loading

✅ Deploy Preview for thingswedo ready!

Name Link
🔨 Latest commit e23e950
🔍 Latest deploy log https://app.netlify.com/sites/thingswedo/deploys/675cb0495bc99d0008afc1d8
😎 Deploy Preview https://deploy-preview-76--thingswedo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@jackcasstlesjones
Merge branch 'main' into style/ui-changes
4f7aee5
@jackcasstlesjones
refactor: move home content to homecontent.tsx
ace72e8
@jackcasstlesjones
refactor: change home styling
91e1242
@jackcasstlesjones
feat: home page
4e4522a
@jackcasstlesjones
style: better display for needs screenshot
40d4563
@jackcasstlesjones
chore: change moods to track in navbar
aadce98
@jackcasstlesjones
style: remove background from decision maker
c4f0b67
@jackcasstlesjones
style: remove background from insights screenshot
d93f4c7
@jackcasstlesjones
refactor: remove styles from globals.css
a980606
@jackcasstlesjones
style: change colour of isnights dates bar
7df9e3c
@jackcasstlesjones
chore: remove deprecated props from home page images
b47414c
@jackcasstlesjones
style: add high z index to navbar
0d3cab5
@jackcasstlesjones
chore: update tests
404d0f1
@jackcasstlesjones
chore: update imports
39267c4
@jackcasstlesjones
chore: update tests
d6e3cbe
@jackcasstlesjones
chore: update homecontents to homecontent
5b2987b
@jackcasstlesjones
style: center homepage content
0c2100c
@jackcasstlesjones
style: add font-light to all p elements, and finish styling home page
e88f54b
@jackcasstlesjones jackcasstlesjones changed the title DO NOT MERGE testing on phone App-wide UI styling Dec 13, 2024
@jackcasstlesjones jackcasstlesjones marked this pull request as ready for review December 13, 2024 20:45
maxitect
maxitect requested changes Dec 13, 2024
View reviewed changes
Copy link
Collaborator

@maxitect maxitect left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks so professional now, thanks for going through the whole UI making it look top notch! Couple of suggestions for accessibility

src/app/toolkit/components/CategoriesBar.tsx Show resolved Hide resolved
src/app/toolkit/add-tool/components/AddToolTags.tsx Show resolved Hide resolved
@maxitect maxitect merged commit 7202afd into main Dec 13, 2024
5 checks passed
@maxitect maxitect deleted the style/ui-changes branch December 13, 2024 22:19
maxitect added a commit that referenced this pull request Dec 13, 2024
@maxitect
chore: merge with main pr #76
9c0d3a5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maxitect maxitect maxitect approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jackcasstlesjones @maxitect