search

IanDarwin / pdfshow

Deliver/present talks from PDFs - A simple PDF presenter oriented towards slideshow decks in PDF form
Other
8 stars 4 forks source link

Fix Nav/Toolbox icons are way too big #30

Closed thetodd closed 3 years ago

thetodd commented 3 years ago

Hey, this pull request fixes #12. I reduced the size of the icons to 32x32 pixels. Also adjusted the width of the navbar itself, as you mentioned in the issue you would have more room for the viewer. To make the navbar smaller I had to rearrange the buttons. Bildschirmfoto von 2020-10-22 22-38-10

IanDarwin commented 3 years ago

And you also unified the icon coloration and added tooltips. Thanks!

thetodd commented 3 years ago

Hey @IanDarwin I'm glad to see you are happy with my PR. Would you mind to add the hacktoberfest-accepted label to this PR, so that this counts for me?

IanDarwin commented 3 years ago

On Fri, Oct 23, 2020 at 04:20:38AM -0700, Florian Schleich wrote:

Hey [1]@IanDarwin I'm glad to see you are happy with my PR. Would you mind to add the hacktoberfest-accepted label to this PR, so that this counts for me?

Done! I think I did it correctly, please check (first hacktoberfest-accepted experience).

-- You are receiving this because you were mentioned. Reply to this email directly, [2]view it on GitHub, or [3]unsubscribe.

References

  1. https://github.com/IanDarwin
  2. https://github.com/IanDarwin/pdfshow/pull/30#issuecomment-715280588
  3. https://github.com/notifications/unsubscribe-auth/ABEKZWQ3WZORKWN27YXF6WTSMFRINANCNFSM4S3XBXBA