implement simple custom background option for menus#469
implement simple custom background option for menus#469Follpvosten wants to merge 1 commit intoMalpenZibo:mainfrom
Conversation
|
|
d06ec47 to
886f604
Compare
|
Ok, for these kinds of changes I would like to open a discussion on improving the styling configuration in the next release (post March) |
886f604 to
b983e86
Compare
b983e86 to
173ea68
Compare
173ea68 to
51bc0cb
Compare
|
Ok. Generally speaking, I'm ok with this change. About the configuration, I have some doubts. I would like to maintain the palette as a 1to1 port of the iced palette. For this reason, the I need to think about this also because I plan to have a similar solution for the status bar surface background in the future. |
|
interesting! being able to set a background for the status bar itself would also solve my problem, perhaps even more elegantly. I hadn't considered that option. From my view, both achieve a very similar goal though: Being able to specify different backgrounds for different major parts of the UI. for now I will rework my existing patch to implement things as requested. Further discussion does indeed sound reasonable though. |
51bc0cb to
a2d4c92
Compare
a2d4c92 to
593e1ac
Compare
|
For now I have kept the But in the public API, I have moved Docs-wise, I have removed the line I had previously (where it was documented directly under palette -> background), and instead created a new section further down in |
|
@Follpvosten the palette rework is largely done. Feel free to check #574 and play around with it |
as per the title.
for my specific usecase, this allows me to style the top bar exactly like the one in GNOME - solid black bar, drop-down menus in grey/gtk colour. it remains backwards comspatible by using an Option and falling back to
background_colorif menu_background was not set in config.I'm open for criticism or input/design suggestions, but also open to just getting a "no" (in which case I'd just maintain a downstream fork for this :)