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

Checkbox "Control feature rendering order" is not honored #22311

Closed
qgib opened this issue Feb 17, 2016 · 5 comments
Closed

Checkbox "Control feature rendering order" is not honored #22311

qgib opened this issue Feb 17, 2016 · 5 comments
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Symbology Related to vector layer symbology or renderers
Milestone

Comments

@qgib
Copy link
Contributor

qgib commented Feb 17, 2016

Author Name: Sebastian Dietrich (Sebastian Dietrich)
Original Redmine Issue: 14323
Affected QGIS version: master
Redmine category:symbology


The checkbox Control feature rendering order implies the user can enable or disable the sorting by the order-by-clause shown next to it. However the order-by-clause is used regardless of the status of this checkbox.

Reproduce:

  • Check the checkbox
  • Define an order by clause
  • Uncheck the checkbox
  • Apply the changes
  • -> Rendering order is controlled by the order-by-clause
@qgib
Copy link
Contributor Author

qgib commented Feb 17, 2016

Author Name: Nyall Dawson (@nyalldawson)


I agree that this is bad UX. It's often desirable to be able to temporarily enable/disable features like sorting without losing the sorting expression.

@qgib
Copy link
Contributor Author

qgib commented Feb 17, 2016

Author Name: Sebastian Dietrich (Sebastian Dietrich)


I agree that this is bad UX.

It is even worse. Note that I had to change the subject and description because the checkbox is completely ignored. Only the order-by-clause determines the rendering order, regardless of the checked-state of the checkbox.

I raised the priority to Blocker because I think a new feature must not come with such a misleading UI.


  • priority_id was changed from Normal to High
  • subject was changed from Checkbox "Control feature rendering order" is checked without user interaction to Checkbox "Control feature rendering order" is not honored

@qgib
Copy link
Contributor Author

qgib commented Feb 17, 2016

Author Name: Sebastian Dietrich (Sebastian Dietrich)


  • priority_id was changed from High to Severe/Regression

@qgib
Copy link
Contributor Author

qgib commented Feb 18, 2016

Author Name: Sebastian Dietrich (Sebastian Dietrich)


See "PR 2809":#2809 for a solution. It is just a quick fix replacing the checkbox with a label and should act as a fall-back plan in case no better solution can be implemented before the upcoming release (7 days from now).


  • pull_request_patch_supplied was changed from 0 to 1

@qgib
Copy link
Contributor Author

qgib commented Feb 18, 2016

Author Name: Nyall Dawson (@nyalldawson)


Fixed in changeset "4285d70b6c85b4600cf3f0e55a8408447272180c".


  • 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! Symbology Related to vector layer symbology or renderers labels May 25, 2019
@qgib qgib added this to the Version 2.14 milestone May 25, 2019
@qgib qgib closed this as completed May 25, 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! Symbology Related to vector layer symbology or renderers
Projects
None yet
Development

No branches or pull requests

1 participant