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

Edit expression button looks silly #16834

Closed
qgib opened this issue Jun 2, 2013 · 5 comments
Closed

Edit expression button looks silly #16834

qgib opened this issue Jun 2, 2013 · 5 comments
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Labeling Related to QGIS map labeling
Milestone

Comments

@qgib
Copy link
Contributor

qgib commented Jun 2, 2013

Author Name: Aren Cambre (Aren Cambre)
Original Redmine Issue: 7977
Affected QGIS version: master
Redmine category:labelling


In the Layer labeling settings dialog, there is text Label this layer with, then a dropdown, then a button with a lowercase Greek epsilon and an ellipsis (...). Please remove the epsilon. It makes the button look overdone, and ... alone is sufficient as it is well-understood to mean that there are more options for the adjacent feature.


@qgib
Copy link
Contributor Author

qgib commented Jun 2, 2013

Author Name: Nathan Woodrow (@NathanW2)


I would rather it say Expression and also have the icon. The Greek epsilon is being used though the UI to mean expression so it is there for consistency reasons. The ... alone is not enough as it doesn't tell the user what that button is for (trouble I have had in the past with students in training)

@qgib
Copy link
Contributor Author

qgib commented Jun 2, 2013

Author Name: Aren Cambre (Aren Cambre)


Sorry, didn't realize the epsilon is being used in other places in QGIS. I think I agree that epsilon + Expression is better here.

@qgib
Copy link
Contributor Author

qgib commented Jun 3, 2013

Author Name: Larry Shaffer (Larry Shaffer)


Sorry, I have to disagree. First off, the issue is an opinion, not a bug, and better suited as a post to the mailing list.

IMO, the use of an icon works better, and is now the standard to represent an Expression. The HIG specifies putting an ellipses for actions that pop open a dialog, which is why the button is labeled 'E...'. Other parts of the GUI don't even go that far (see new attribute table).

Having just an ellipsis generally represents opening a file browsing dialog of some kind. That is clearly not the case here, so more is needed.

The button already has a tool tip that says 'Edit expression'. Also ALL of the new labeling data defined buttons use the epsilon symbol to represent when an expression is set. Users will learn that the epsilon represents expressions or expression functions, possibly after just one use. IMO, there should be no change, except maybe how the epsilon is styled.

Initially, I thought just an 'E' was not enough. "I proposed an '(E)' instead":http://drive.dakotacarto.com/qgis/expression_ls.png, however, though it was accepted, for whatever reason it was not adopted.


  • 5829 was configured as expression_ls.png

@qgib
Copy link
Contributor Author

qgib commented Jul 21, 2013

Author Name: Nathan Woodrow (@NathanW2)


  • resolution was changed from to wontfix

@qgib
Copy link
Contributor Author

qgib commented Jul 21, 2013

Author Name: Nathan Woodrow (@NathanW2)


  • status_id was changed from Open to Closed

@qgib qgib added Bug Either a bug report, or a bug fix. Let's hope for the latter! Labeling Related to QGIS map labeling labels May 24, 2019
@qgib qgib added this to the Version 2.0.0 milestone May 24, 2019
@qgib qgib closed this as completed May 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Labeling Related to QGIS map labeling
Projects
None yet
Development

No branches or pull requests

1 participant