-
Notifications
You must be signed in to change notification settings - Fork 163
Make popover styling consistent #1567
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
base: master
Are you sure you want to change the base?
Conversation
|
Pop-overs still have different padding than drop-downs, so there's still some inconsistency here. I want to make sure we know what direction we're going with this. This is what the pop-over currently looks like on the VPN page:
This is what it used to look like on the Bluetooth page:
This is what it will now look like on the Bluetooth page:
This is what drop-downs still look like:
@pop-os/ux Is this the direction we want to make the change, or would it be better to make the VPN pop-over match what the others currently look like (which is more similar to the drop-downs)? This topic is very similar to what was being discussed in pop-os/libcosmic#1042 (comment) regarding drop-down styling across different components. |
|
This was mainly because the popovers on the keyboard and region pages were quite different and didn't feel nice. |
|
Sorry for the delayed response. The dropdown menus that open after clicking on "view more" icon should all follow the dropdown menu styling for now: https://www.figma.com/design/SkAtS5qlVOGQC0n895AHzX/Design-System--WIP-?node-id=11667-220337&t=gEV0p3UotnX2xe1B-4. It should also be a widget from libcosmic. That way if we decide to change the styling of the dropdowns later to match other menus we only need to do it in the library. At least, ideally. |
Makes all popovers use the same styling, with hover highlights.
|
I've updated this for testing with pop-os/libcosmic#1053 (libcosmic needs to be patched with a local clone of that in Cargo.toml to properly test). |
I think the figma reference design needs a retake. Nothing justifies the check icon as a suffix. |









Makes all popovers use the same styling, with hover highlights (matching those on the VPN page currently).
Also removes some unnecessary Vecs.