Add Lubuntu Round Openbox Theme and change background and hover color for in menu search
Details
Tst if in menu search has the new hover and backgroun colors, same as menu
Diff Detail
- Repository
- rART Artwork
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Event Timeline
As I said before, I trust you fully as far as theming is concerned. So I'm not too worried about the aesthetics of the theme.
I also don't consider myself much of an expert on how one does the theming, either. But I have two observations in regards to that subject:
- It seems that the main menu background was somehow based off of the text palette rather than being hardcoded, which, I'm thinking, might make it more dynamic.
- The main menu selected item goes from being an rgb (decimal) value to being an html (hexidecimal) one. I'm sure as far as the end result is concerned, this doesn't ultimately matter, but perhaps consistency might not be wise?
You can tell me if I don't know what I'm talking about :)
As far as the theme is concerned, my concerns are:
- The commented lines which should be removed, unless we're trying to expose some options to allow people to change, in which case, they should be documented.
- It should have a Lubuntu copyright.
I guess those things relate to our current theme, so those should probably get fixed, too…
- It seems that the main menu background was somehow based off of the text palette rather than being hardcoded, which, I'm thinking, might make it more dynamic.
It sounds good in theory but in practice I haven't been able to make it work well because of our strange combination dark/light. palette(text) gives the light background https://phab.lubuntu.me/file/data/ttrldxvuelxtrwlrkxqd/PHID-FILE-rewnjzd6z5bnokqc2cbs/image.png , palette gives black https://phab.lubuntu.me/file/data/f7xfnxptzrtpbccbv6aa/PHID-FILE-t3uhfy727jqnveam45z7/image.png . If same result https://phab.lubuntu.me/file/data/5ca5lmzalxicymra5icf/PHID-FILE-p6c4vzko4e7mmmoivcxp/image.png could be achieve with palette I would prefer palette, but I'm not having been able to do it. I'm no expert on colors so if there are better options I'm willing to use them.
- The main menu selected item goes from being an rgb (decimal) value to being an html (hexidecimal) one. I'm sure as far as the end result is concerned, this doesn't ultimately matter, but perhaps consistency might not be wise?
I used the same consistency of the original file https://phab.lubuntu.me/source/lubuntu-artwork/browse/ubuntu%252Ffocal/src/usr/share/lxqt/themes/Lubuntu%20Arc/lxqt-panel.qss in which those backgrounds are rgb. We can change all of them, but I don't know if that has been done for a reason or not.
- The commented lines which should be removed, unless we're trying to expose some options to allow people to change, in which case, they should be documented.
In xbm lines starting with # are definitions, not comments https://en.wikipedia.org/wiki/X_BitMap
- It should have a Lubuntu copyright.
You mean change this?
Copyright (C) 2015 Dino Duratović <dinomol at mail dot com>
with this?
Copyright (C) 2020 Lubuntu
Well maybe I'm backpedalling on all of that:
- if using the palette doesn't work, oh well
- in regards to color types, probably better not to fix what's not broken ☺
- I meant the comments in the themerc, but…
- …is this just a copy of the original?
- …is this just a copy of the original?
99% is, except the close.xbm and in themerc the lines:
window.active.button.close.unpressed.image.color:#BA63D1
window.inactive.button.unpressed.image.color: #7F8388
there are 3 commented lines I added 47-49 I did it so long ago I forgot. I can erase them, shoul I erase the other commented too?
Yeah I guess it should have something similar to the original where this was inspired by so-and-so and we modified it and so it should have a Lubuntu copyright. I wonder who Dino Duratović is and where the original came from.
I take my acceptance back but just a little. It is a bit picky but I know you are getting ready to apply for developer (hint, hint). Some of the lintian items in P64 should probably be addressed. Most notably the Standards-Version and probably the whitespace in d/control. @The_LoudSpeaker will want to address the grub theme issues at some point too.
debian/changelog | ||
---|---|---|
5 | Throw a Bumped Standards-Version to 4.5.0, no changes needed. here and LGTM |
- changed copyright, removed some commented lines, bumped standard versions apply new debhelper compat standard