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

Add consistency to attribute's dialogs #2327

Merged
merged 5 commits into from
Nov 21, 2015
Merged

Conversation

DelazJ
Copy link
Contributor

@DelazJ DelazJ commented Sep 20, 2015

This PR use same terms in the different attribute's dialogs. This replace :

  • "New column" and "Delete column" buttons label in attribute table dialog and layer's property> field's page by the terms "New field" and "Delete field". Their corresponding dialogs title are also renamed.
  • "Field" is used rather than "attributes" in dialogs that create a new OGR vector layer or a new spatialite one.
  • the attribute "width" of field is replaced by "length"
  • fixes some descriptions and typos in attribute's form customization

sorry; Could not build qgis so screenshots are from QtDesigner.
qgsnewvectorlayerdialogbase
qgsaddattributetabledialog
qgsattributetabledialog

partly fixes #9464
See comments in the issue report.

@m-kuhn
Copy link
Member

m-kuhn commented Sep 21, 2015

For reference http://hub.qgis.org/issues/9464

@m-kuhn
Copy link
Member

m-kuhn commented Sep 21, 2015

@DelazJ Please edit the top message to include details about what this pull request introduces and attach screenshots if useful. This makes it much easier for reviewers.

@anitagraser @nyalldawson @nirvn @NathanW2

Calling the UX department

@DelazJ
Copy link
Contributor Author

DelazJ commented Sep 24, 2015

@m-kuhn , sorry for the delay. More comments added now.

To fully solve the issue and the thoughts that have emerged from that report, I'd like to know:

  • Are there rules about Title casing of dialog's title?
  • What are the correspondance between type and provider type fields in "add a new column" dialog and type and type name in layer's properties > fields tab. Are they about same things? And which terms to keep for consistency?

@m-kuhn
Copy link
Member

m-kuhn commented Sep 25, 2015

@DelazJ thanks for the update.

Not sure about a rule, I'd personally go for capitalized dialog titles.

provider type is the native raw data type from the provider.
Since all providers have (slightly) different types, they need to be converted to something more generic we can use inside QGIS. These generic types are the type.
So they both have their purpose.

+1 for Width -> Length

I think I remember a discussion (on the mailing list?) about field vs. attribute and that somebody ( @wonder-sk ? ) to argue that the term attribute should be used on layer/table level and field at the feature/row level. Not sure what others think about it. Personally I tend towards consistency but I'd like to wait for other comments.

@NathanW2 NathanW2 added this to the 2.14 milestone Nov 5, 2015
@NathanW2
Copy link
Member

NathanW2 commented Nov 5, 2015

@m-kuhn I'm a fan of consistency. IMO attribute == field so we just pick on and go with it at least for the UI.

@carlospsig
Copy link

Go for consistency!

(Matthias at the keyboard)
El dia 06/11/2015 0:03, "Nathan Woodrow" notifications@github.com va
escriure:

@m-kuhn https://github.com/m-kuhn I'm a fan of consistency. IMO attribute
== field so we just pick on and go with it at least for the UI.


Reply to this email directly or view it on GitHub
#2327 (comment).

@anitagraser
Copy link
Member

Great, +1 for consistency!

If you are already touching the attribute table dialog: Could we add undo and redo buttons like we have in the advanced editing toolbar? Otherwise the only way to undo, e.g. adding a field, is to stop editing and discard all previous changes.

@@ -1617,7 +1617,7 @@
<widget objectName="classificationLabel" class="QLabel" label="Displays combo box containing values of attribute used for classification."/>
</widget>
<widget objectName="lineEditPage" class="QWidget" label="">
<widget objectName="lineEditLabel" class="QLabel" label="Simple edit box. This is the default editation widget."/>
<widget objectName="lineEditLabel" class="QLabel" label="Simple edit box. This is the default edition widget."/>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you change that to "edit widget"

@michaelkirk
Copy link
Contributor

👍 This is great, thank you!

m-kuhn added a commit that referenced this pull request Nov 21, 2015
Add consistency to attribute's dialogs
@m-kuhn m-kuhn merged commit 504c252 into qgis:master Nov 21, 2015
@DelazJ DelazJ deleted the ConsistentTerms2 branch April 4, 2016 15:25
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

6 participants