Frontmatter
| id | 4612 |
| title | Variables for popover menu |
| state | Closed |
| labels | enhancement |
| assignees | [] |
| createdAt | Jul 31, 2023, 6:05 PM |
| updatedAt | Aug 3, 2023, 11:28 PM |
| githubUrl | https://github.com/neomjs/neo/issues/4612 |
| author | mxmrtns |
| commentsCount | 4 |
| parentIssue | null |
| subIssues | [] |
| subIssuesCompleted | 0 |
| subIssuesTotal | 0 |
| blockedBy | [] |
| blocking | [] |
| closedAt | Aug 3, 2023, 11:28 PM |
Variables for popover menu
tobiu
Aug 1, 2023, 12:39 AM
i guess i should have mentioned that menu.List is extending list.Base which already has some of them :)
well, unless we do want different styling for lists when they are inside floating containers.
tobiu
Aug 1, 2023, 12:50 AM
added around half of them. let's connect tomorrow to figure out, if modifying list.Base instead is smarter.
tobiu
Aug 3, 2023, 11:13 PM
i assume that "pressed" should mean "selected". unless you want a ripple effect or a different state onMouseDown (which is different to the selected look and vanishes onMouseUp).
tobiu
Aug 3, 2023, 11:28 PM
i think i got them all now.
tobiu closed this issue on Aug 3, 2023, 11:28 PM
For styling this component, we would need the following variables.
Menu Item
height
padding
gap (something to control the distance between the icons and text)
font-familiy, color, font-weight, line-height, text-transform
font-color and icon color for disabled state
background-color
background-color-hover
background-color-pressed
background-color-disabled
-outlined-focused