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

Closing the block inserter decrements block type impressions #3802

Merged

Conversation

dcalhoun
Copy link
Member

@dcalhoun dcalhoun commented Aug 5, 2021

Relates to WordPress/gutenberg#33907. Closing the block inserter will decrement the block type impression counts. This results in blocks that are marked as "new" to slowly lose the designation as a user utilizes the block inserter over time.

To test: See WordPress/gutenberg#33906.

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered if this change warrants user-facing release notes more info and have added them to RELEASE-NOTES.txt if necessary.

@dcalhoun dcalhoun self-assigned this Aug 5, 2021
@dcalhoun dcalhoun added the [Type] Enhancement Improves a current area of the editor label Aug 5, 2021
@dcalhoun dcalhoun modified the milestones: 1.59.0 (18.0), 1.60.0 (18.1) Aug 5, 2021
@dcalhoun dcalhoun marked this pull request as ready for review August 5, 2021 18:10
@dcalhoun dcalhoun requested a review from mchowning August 5, 2021 18:11
Copy link
Contributor

@mchowning mchowning left a comment

Choose a reason for hiding this comment

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

@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Aug 5, 2021

Wanna run full suite of Android and iOS UI tests? Click here and 'Approve' CI job!

…e into update/closing-inserter-decrements-block-type-impressions
@dcalhoun dcalhoun enabled auto-merge August 6, 2021 18:28
@dcalhoun dcalhoun merged commit 157892a into develop Aug 6, 2021
@dcalhoun dcalhoun deleted the update/closing-inserter-decrements-block-type-impressions branch August 6, 2021 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Enhancement Improves a current area of the editor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants