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

Field properties (comments and alias) update #8795

Merged
merged 3 commits into from
Feb 16, 2024

Conversation

selmaVH1
Copy link
Collaborator

fixes #8166

@DelazJ DelazJ added the backport release_3.34 On merge create a backported pull request to 3.34 label Jan 22, 2024
@@ -2068,7 +2068,7 @@ fields related to the layer and helps you organize them.
The layer can be made :ref:`editable <editingvector>` using the |toggleEditing|
:sup:`Toggle editing mode`. At this moment, you can modify its structure using
the |newAttribute| :sup:`New field` and |deleteAttribute| :sup:`Delete field`
buttons.
buttons.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
buttons.
buttons.

@@ -2077,7 +2077,8 @@ depending on the GDAL version.
If set in the underlying data source or in the :ref:`forms properties
<configure_field>`, the field's alias is also displayed. An alias is a human
readable field name you can use in the feature form or the attribute table.
Aliases are saved in the project file.
Aliases are saved in the project file. Currently supported for OGR formats
with field comment capabilities such as GeoPackage or ESRI File Geodatabase.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think a feature here is that you now have a "comment" box (along with alias) you can fill when creating a field for a layer that supports it. Below, a gpkg.
add_comment
Btw, why are the lists of formats different? should we not have the same list as in lines 2096-97?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I added new explanation, hope this is more clarified. I used information from changelog.

@selmaVH1 selmaVH1 merged commit e97e713 into qgis:master Feb 16, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport release_3.34 On merge create a backported pull request to 3.34
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cleanup 'comment' handling in new field dialog, add support for alias storage (Request in QGIS)
2 participants