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

panel-config: Add border_width property #283

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ryanabx
Copy link
Contributor

@ryanabx ryanabx commented Nov 2, 2024

This commit adds a new property to the panel config, which allows users to specify a border for the panel.

The default is 0.0, and we don't expose this property in cosmic-settings.

Design approval pending :)

Example of a border_width of 1.0:

screenshot-2024-11-02-05-36-50

This commit adds a new property to the panel config, which allows users to specify a border for the panel.

Signed-off-by: Ryan Brue <[email protected]>
Signed-off-by: Ryan Brue <[email protected]>
@mmstick
Copy link
Member

mmstick commented Dec 31, 2024

Needs approval by @maria-komarova. Probably fine since it's not enabled by default.

@maria-komarova
Copy link

It's all right to add the property as long as it doesn't change the defaults.

@wash2
Copy link
Collaborator

wash2 commented Jan 2, 2025

There seems to be a bug relating to how the border is drawn, as it doesn't line up correctly when I've tested it.

@ryanabx
Copy link
Contributor Author

ryanabx commented Jan 22, 2025

There seems to be a bug relating to how the border is drawn, as it doesn't line up correctly when I've tested it.

Could be because I used the iced border_width property? Maybe that's an outer border instead of an inner border. If so, there might need to be some size adjustments to accommodate, but that might affect the size of applets? Maybe not, /shrug

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.

4 participants