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.

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

Dropdown menu #344

Merged
merged 3 commits into from
Nov 4, 2023
Merged

Dropdown menu #344

merged 3 commits into from
Nov 4, 2023

Conversation

mruwnik
Copy link
Collaborator

@mruwnik mruwnik commented Nov 1, 2023


.menu-link:last-child {
border-bottom: none;
}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I really hope someone will come along and do proper styling of everything, so I don't have to...

Copy link
Collaborator

Choose a reason for hiding this comment

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

IMHO if there is no link to GH issue description and/or Figma, then there is no "proper" styling to implement 🤷

Copy link
Collaborator

Choose a reason for hiding this comment

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

From #285 hidden to the far right of linked Figma file
Screenshot 2023-11-02 at 1 19 18 AM
Screenshot 2023-11-02 at 1 21 27 AM

d="M283.7 127.6h-60.6c-30.4 0-55.1 24.7-55.1 55 0 4.4 3.6 8 8 8s8-3.6 8-8c0-21.5 17.5-39.1 39.1-39.1h60.6c24.4 0 44.3 19.9 44.3 44.3 0 16.9-9.4 32.1-24.6 39.7L252.3 254c-2.6 1.4-4.3 4.2-4.3 7.1V288c0 4.4 3.6 8 8 8s8-3.6 8-8v-21.9l46.7-24.2c20.5-10.3 33.3-31 33.3-54 0-33.3-27.1-60.3-60.3-60.3zM256 328c-8.8 0-16 7.2-16 16s7.2 16 16 16 16-7.2 16-16-7.2-16-16-16zm0-296.9C114.6 31.1.9 124.2.9 239.1c0 49.6 21.4 95 57 130.7-12.5 50.4-54.3 95.3-54.8 95.8-2.2 2.3-2.8 5.7-1.5 8.7s4.1 4.8 7.3 4.8c66.3 0 116-31.8 140.6-51.4 32.7 12.3 69 19.4 107.4 19.4 141.4 0 255.1-93.1 255.1-208s-114.6-208-256-208zm0 400c-35.1 0-69.4-6.2-101.8-18.4l-8.5-2.3-7.1 5.7c-20.2 16.1-60.1 42-112.7 47.1 14.4-18.3 37.4-51.7 46.6-88.6l2.2-8.8-6.4-6.4C34.1 325.1 16 283.8 16 239.1c0-105.9 107.7-192 240-192s240 86.1 240 192-107.7 192-240 192z"
style={{
fill: '#333',
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

this now uses the default fill colour, which will handle light/dark mode gracefully

@mruwnik
Copy link
Collaborator Author

mruwnik commented Nov 1, 2023

should I just deploy this to a test server, rather than uploading millions of pictures?

@Aprillion
Copy link
Collaborator

should I just deploy this to a test server

yes

Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm not sure about the reversal of "asking for help" icon, the usual meaning is that the user asks for help and the site is helpful ... here the site is asking for help and the user is supposed to be helpful 🤔

How is the user supposed to know that meaning before clicking on the icon? Was this discussed on some meeting? What was the answer on that meeting, please?

Copy link
Collaborator

Choose a reason for hiding this comment

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

seeing it in action on https://stampy-ui.ai688.workers.dev/ the text Help Out is more visible, so maybe this is OK after all

Aprillion
Aprillion previously approved these changes Nov 2, 2023
@mruwnik mruwnik merged commit 5304a21 into master Nov 4, 2023
1 check passed
@mruwnik mruwnik deleted the dropdown-menu branch November 4, 2023 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants