muriloventuroso / pdftricks

A simple, efficient application for small manipulations in PDF files using Ghostscript.
91 stars 12 forks source link

Icons Redesign #38

Closed Fatih20 closed 5 years ago

Fatih20 commented 5 years ago

The previous icons did not comply within the eOS HIG. The "tool" icon at the corner was really out of line with eOS HIG guidlines (see the pun?) regarding the implementation of pictogram. It wasn't extruded nor inset. Furthermore, it also doesn't comply by the eOS HIG rules regarding highlight and shadows for the tool in the corner. I've look into As for the other app icons, it's too on the nose regarding to what the each icons represent. I've look into your apps and there is already an explanation to what the button does. In that case, the icon doesn't have to be super clear and detailed to what it represents. Instead, they should be moe abstract and be more focused on being aesthetically pleasing. For obvious reason I have mentioned.

Fatih20 commented 5 years ago

Which parts are too transparent? Do you think I should add a white rectangle as base shape below everything?

NathanBnm commented 5 years ago

I mean, on most of the icons there is the "PDF" symbol in background, on which you added a magic wand, or an arrow.

And for the merge, and cut icons you removed this background. I'm worried about how it will be rendered in the UI.

I think you should keep the design you made but with the background added back.

Do you understand what I mean?

Fatih20 commented 5 years ago

Oh yeah, there were some technical issues on my side. Will be fixing that soon. Thanks for pointing that out.

muriloventuroso commented 5 years ago

I agree with @NathanBnm . With the transparent icons the aim of the icon is lost, it is difficult to identify the action.

Fatih20 commented 5 years ago

No, it's not an aim. It's a technical issue, there's supposed to be a PDF icon there.

Fatih20 commented 5 years ago

There, fixed it.

Fatih20 commented 5 years ago

Can you please credit me for the icon? Thank you!

muriloventuroso commented 5 years ago

Of course, I will provide new screenshots and update the credits.

Fatih20 commented 5 years ago

Credit me in the release notes, please. :)