In D9549#184458, @ngraham wrote:The other proposed change is D8232, by @rpelorosso, who has expressed positive sentiments about this one. @rpelorosso, would you mind closing D8232?
- Queries
- All Stories
- Search
- Advanced Search
Feed Advanced Search
Advanced Search
Advanced Search
Jan 2 2018
Jan 2 2018
Closing this revision in favor of D9549.
Dec 29 2017
Dec 29 2017
These new shadows look amazing! :D
Nov 10 2017
Nov 10 2017
rpelorosso added a comment to D8361: Add option to change Oxygen's windows decoration shadow intensity.
In D8361#165581, @abetts wrote:In D8361#165580, @ngraham wrote:Yeah, maybe a combobox toggling between Glow and Shadow styles/blending modes would be more intuitive here.
That's what I mean.
Oct 23 2017
Oct 23 2017
rpelorosso added a comment to D8361: Add option to change Oxygen's windows decoration shadow intensity.
@ngraham Sorry to bother you, but I'm fairly new to this process. As this hasn't gotten any attention, is there any other group that might want to review this beside VDG? (or is it just that there's no interest in something like this? Thanks!
Oct 19 2017
Oct 19 2017
rpelorosso added a reviewer for D8361: Add option to change Oxygen's windows decoration shadow intensity: VDG.
Oct 18 2017
Oct 18 2017
rpelorosso retitled D8361: Add option to change Oxygen's windows decoration shadow intensity from Add option to change windows shadow intensity to Add option to change Oxygen's windows decoration shadow intensity.
rpelorosso updated the summary of D8361: Add option to change Oxygen's windows decoration shadow intensity.
Oct 14 2017
Oct 14 2017
I didn't want to stop mentioning that, unless I'm missing something, it looks like breeze *window* shadows are not taking into consideration the Screen Scaling settings, so basically they don't scale if that value is increased, making the shadows look small in 4k screens with screen scaling. I haven't yet created a bug report because 1- I want to really make sure this is an issue, and 2- I want to have a look at it if that's really the case.
In D8232#155503, @ngraham wrote:
In D8232#155519, @alake wrote:Ok, I agree now that there shouldn't be an option. I think it is fair to say that the contrast on the left side of overlapping windows could be improved. The proposed compromise solution actually looks decent too and it keeps the lighting reasonably consistent. The VDG and Hugo are discussing. Oh and in case it is sometimes forgotten, thanks for submitting the patch. :-)
In D8232#155436, @ngraham wrote:I completely agree, @rkflx. For just this reason, 10 years ago Apple changed the window shadows to be super super gigantic: https://arstechnica.com/gadgets/2007/10/mac-os-x-10-5/3/
It was a bit controversial at first, but eventually people really liked it for exactly the reason you indicate: it provided a great sense of depth. I always make Breeze shadows as huge as possible to try to emulate this. Programmer-types like us often underestimate just how hard it is for many people so know which window is in the active one. Anything we can do ti make it more obvious, without becoming garish, IMHO is worth it.
In D8232#155278, @hpereiradacosta wrote:4/ note that oxygen has centered shadows everywhere (decoration, menus and buttons).
Thank you for your comments! I've made the requested changes.
- Use selectionNotEmpty instead of explicit comparison
- Use urlList() directly instead of using a variable to hold it
I'm glad you like it, and that it is a change that was planned for the future. It is definitely a feature I was missing. I've updated the option's title. Also, the new option is included in the regular File menu, I've attached a screenshot of it.
Changed option title from "Show in File Browser" to "Open Containing Folder"
Oct 10 2017
Oct 10 2017
I added a screenshot to the patch summary, here's the link to it.
Aligned the "Center Shadow" checkbox to the left