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

Clearing Multiple QGIS Warnings takes inordinately long #29698

Closed
qgib opened this issue Apr 18, 2019 · 0 comments · Fixed by #36696
Closed

Clearing Multiple QGIS Warnings takes inordinately long #29698

qgib opened this issue Apr 18, 2019 · 0 comments · Fixed by #36696
Assignees
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! GUI/UX Related to QGIS application GUI or User Experience

Comments

@qgib
Copy link
Contributor

qgib commented Apr 18, 2019

Author Name: Brett Carlock (@Saijin-Naib)
Original Redmine Issue: 21883
Affected QGIS version: 3.6.1
Redmine category:gui


In the attached image there are some 7782 QGIS warnings that need clearing. I click the button for Clear All, and I have to wait minutes for them all to clear. This process should be instant.


@qgib qgib added Bug Either a bug report, or a bug fix. Let's hope for the latter! GUI/UX Related to QGIS application GUI or User Experience labels May 25, 2019
@nyalldawson nyalldawson self-assigned this May 25, 2020
nyalldawson added a commit to nyalldawson/QGIS that referenced this issue May 25, 2020
Avoid unnecessary calls to setStyleSheet, which is slow when many
message bar items exists

Refs qgis#29698
nyalldawson added a commit to nyalldawson/QGIS that referenced this issue May 25, 2020
The previous size limitation code was rather broken, and didn't apply
to all messages.

Add a refined approach which automatically removes the oldest message,
prioritising lower impact messages (i.e. a "success" message will
always be removed before a "warning")

Also fixes a quasi leak where items could be added to the bar and never
deleted.

Fixes qgis#29698
nyalldawson added a commit that referenced this issue May 26, 2020
Avoid unnecessary calls to setStyleSheet, which is slow when many
message bar items exists

Refs #29698
nyalldawson added a commit that referenced this issue May 26, 2020
The previous size limitation code was rather broken, and didn't apply
to all messages.

Add a refined approach which automatically removes the oldest message,
prioritising lower impact messages (i.e. a "success" message will
always be removed before a "warning")

Also fixes a quasi leak where items could be added to the bar and never
deleted.

Fixes #29698
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! GUI/UX Related to QGIS application GUI or User Experience
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants