Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SVG for pan icons #698

Merged
merged 1 commit into from Aug 15, 2013
Merged

SVG for pan icons #698

merged 1 commit into from Aug 15, 2013

Conversation

3nids
Copy link
Member

@3nids 3nids commented Jul 4, 2013

following discussion on the mailing list.

replace pan icon by a hand, and accord panToSelected to already used colors

See the chosen icons:
panicons2

@haubourg
Copy link
Member

haubourg commented Jul 5, 2013

Hi Denis,
I really like all your new icons. My only remarks is for pan hand. The one you use has a pointing finger, which let more think of a pointer cursor. A classical hand would be more meaningfull.
I found some examples: http://docs.autodesk.com/ACD/2010/ENU/AutoCAD%202010%20User%20Documentation/images/PTDCPM/Gator-All/English/1_3dpan_32.png
http://www.tourcoing.fr/patrimoine/xpo1906/hands/hands2/hand.gif
Cheers
Régis

@3nids
Copy link
Member Author

3nids commented Jul 5, 2013

I like the second one, I'll try to find a SVG.

@3nids
Copy link
Member Author

3nids commented Jul 8, 2013

OK, I made the finder shorter.
pan3

What do you think?

@3nids
Copy link
Member Author

3nids commented Jul 8, 2013

Also, I wonder why this icon is used for touch:
mactiontouch

Don't we want to use something like this:
mactiontouch2

@ghost ghost assigned dakcarto Aug 7, 2013
timlinux added a commit that referenced this pull request Aug 15, 2013
@timlinux timlinux merged commit 69d80b9 into qgis:master Aug 15, 2013
@3nids 3nids deleted the svgpan branch August 15, 2013 11:44
@mbernasocchi
Copy link
Member

@3nids Regarding the touch icon, the second one was used for a while, than changed because we had no hands (and it looked a bit cartoon like to me). but I don't like the one used for touch now and I'm no good graphic artist. so I'm open to whichever.

@haubourg
Copy link
Member

@mbernasocchi Agreed. What about something like this ?
mti_64
http://icy.bioimageanalysis.org/image.php?idAttach=681

@mbernasocchi
Copy link
Member

@haubourg already better

@3nids
Copy link
Member Author

3nids commented Aug 20, 2013

for the touch icon, if you want to I can integrate this one.

@mbernasocchi
Copy link
Member

@3nids I'm ok with it. thanks

@timlinux
Copy link
Member

My problem is that these icons (e.g. @haubourg 's example above are completely out of keeping with the rest of the icons....

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants