rpelorosso (rodrigo pelorosso)
User

Projects

User does not belong to any projects.

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Sunday

  • Clear sailing ahead.

User Details

User Since
Oct 10 2017, 5:07 AM (341 w, 3 d)
Availability
Available

Recent Activity

Jan 2 2018

rpelorosso added a comment to D9549: Horizontally center shadows and make them bigger.

The other proposed change is D8232, by @rpelorosso, who has expressed positive sentiments about this one. @rpelorosso, would you mind closing D8232?

Jan 2 2018, 2:07 PM · Plasma
rpelorosso abandoned D8232: Add option to center shadow .

Closing this revision in favor of D9549.

Jan 2 2018, 2:05 PM · Plasma

Dec 29 2017

rpelorosso added a comment to D9549: Horizontally center shadows and make them bigger.

These new shadows look amazing! :D

Dec 29 2017, 6:19 PM · Plasma

Nov 10 2017

rpelorosso added a comment to D8361: Add option to change Oxygen's windows decoration shadow intensity.

Yeah, maybe a combobox toggling between Glow and Shadow styles/blending modes would be more intuitive here.

That's what I mean.

Nov 10 2017, 12:34 AM · Plasma

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 23 2017, 9:48 PM · Plasma

Oct 19 2017

rpelorosso added a reviewer for D8361: Add option to change Oxygen's windows decoration shadow intensity: VDG.
Oct 19 2017, 1:03 PM · Plasma

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.
Oct 18 2017, 7:26 PM · Plasma
rpelorosso updated the summary of D8361: Add option to change Oxygen's windows decoration shadow intensity.
Oct 18 2017, 7:23 PM · Plasma
rpelorosso created D8361: Add option to change Oxygen's windows decoration shadow intensity.
Oct 18 2017, 7:21 PM · Plasma

Oct 14 2017

rpelorosso added a comment to D8232: Add option to center shadow .

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.

Oct 14 2017, 10:26 PM · Plasma
rpelorosso added a comment to D8232: Add option to center shadow .

Check out the large centered shadows in ElementaryOS. They're goregeous:

Oct 14 2017, 10:26 PM · Plasma
rpelorosso added a comment to D8232: Add option to center shadow .
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. :-)

Oct 14 2017, 10:18 PM · Plasma
rpelorosso added a comment to D8232: Add option to center shadow .

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.

Oct 14 2017, 7:29 PM · Plasma
rpelorosso added a comment to D8232: Add option to center shadow .

4/ note that oxygen has centered shadows everywhere (decoration, menus and buttons).

Oct 14 2017, 4:01 PM · Plasma
rpelorosso added a comment to D8282: Add option to show file in File Manager.

Thank you for your comments! I've made the requested changes.

Oct 14 2017, 2:59 PM
rpelorosso updated the diff for D8282: Add option to show file in File Manager.
  • Use selectionNotEmpty instead of explicit comparison
  • Use urlList() directly instead of using a variable to hold it
Oct 14 2017, 2:55 PM
rpelorosso added a comment to D8282: Add option to show file in File Manager.

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.

Oct 14 2017, 4:13 AM
rpelorosso updated the diff for D8282: Add option to show file in File Manager.

Changed option title from "Show in File Browser" to "Open Containing Folder"

Oct 14 2017, 4:10 AM
rpelorosso updated the test plan for D8282: Add option to show file in File Manager.
Oct 14 2017, 1:26 AM
rpelorosso created D8282: Add option to show file in File Manager.
Oct 14 2017, 1:25 AM

Oct 10 2017

rpelorosso added a comment to F5381122: image.png.

I added a screenshot to the patch summary, here's the link to it.

Oct 10 2017, 7:24 PM
rpelorosso updated the diff for D8232: Add option to center shadow .

Aligned the "Center Shadow" checkbox to the left

Oct 10 2017, 2:01 PM · Plasma
rpelorosso added a reviewer for D8232: Add option to center shadow : Breeze.
Oct 10 2017, 5:28 AM · Plasma
rpelorosso created D8232: Add option to center shadow .
Oct 10 2017, 5:20 AM · Plasma