Skip to content

fix(editor): DP-179668 some toolbar buttons not returning focus to editor#1117

Merged
Maysam Kangarani Farahani (maysamkf) merged 1 commit intostagingfrom
DP-179668
Mar 11, 2026
Merged

fix(editor): DP-179668 some toolbar buttons not returning focus to editor#1117
Maysam Kangarani Farahani (maysamkf) merged 1 commit intostagingfrom
DP-179668

Conversation

@maysamkf
Copy link
Contributor

@maysamkf Maysam Kangarani Farahani (maysamkf) commented Mar 10, 2026

Some toolbar buttons not returning focus to editor

Obligatory GIF (super important!)

Obligatory GIF

🛠️ Type Of Change

These types will increment the version number on release:

  • Fix
  • Feature
  • Performance Improvement
  • Refactor

📖 Jira Ticket

https://dialpad.atlassian.net/browse/DP-179668

📖 Description

  • Toolbar popover/dropdown buttons now accept an optional callback via close(cb) that is called after the popover/dropdown fully closes
  • close() is passed focusEditor as the callback so the editor regains focus after close

💡 Context

When selecting a value from the font style, font size, or variable popover, focus returns to the toolbar button instead of the editor.
This is not expected as all other editor toolbar buttons return focus to the editor.

📝 Checklist

For all PRs:

  • I have ensured no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).
  • I have reviewed my changes.
  • I have added all relevant documentation.
  • I have considered the performance impact of my change.

For all Vue changes:

  • I have added / updated unit tests.
  • I have validated components with a screen reader.
  • I have validated components keyboard navigation.

📷 Screenshots / GIFs

Observed bug:

Screen.Recording.2026-03-10.at.3.53.20.PM.mov

Fix:

Screen.Recording.2026-03-10.at.3.38.38.PM.mov

@github-actions
Copy link
Contributor

Please add either the visual-test-ready or no-visual-test label to this PR depending on whether you want to run visual tests or not.
It is recommended to run visual tests if your PR changes any UI. ‼️

@maysamkf Maysam Kangarani Farahani (maysamkf) added the no-visual-test Add this tag when the PR does not need visual testing label Mar 10, 2026
@github-actions
Copy link
Contributor

✔️ Deploy previews ready!
😎 Dialtone documentation preview: https://dialtone.dialpad.com/deploy-previews/pr-1117/
😎 Dialtone-vue preview: https://dialtone.dialpad.com/vue/deploy-previews/pr-1117/

@maysamkf Maysam Kangarani Farahani (maysamkf) changed the title fix(editor): some toolbar buttons not returning focus to editor fix(editor): DP-179668 some toolbar buttons not returning focus to editor Mar 10, 2026
@maysamkf Maysam Kangarani Farahani (maysamkf) marked this pull request as ready for review March 10, 2026 22:59
@maysamkf Maysam Kangarani Farahani (maysamkf) merged commit 67a2a64 into staging Mar 11, 2026
28 of 31 checks passed
Brad Paugh (braddialpad) pushed a commit that referenced this pull request Mar 12, 2026
## [3.215.1](dialtone-vue/v3.215.0...dialtone-vue/v3.215.1) (2026-03-12)

### Bug Fixes

* **Editor:** DP-179668 some toolbar buttons not returning focus to editor ([#1117](#1117)) ([67a2a64](67a2a64))
* **Popover:** DP-175009 revert visibility CSS that caused tippy-box click stealing ([#1119](#1119)) ([82ffcb6](82ffcb6))
Brad Paugh (braddialpad) pushed a commit that referenced this pull request Mar 12, 2026
## [9.171.1](dialtone/v9.171.0...dialtone/v9.171.1) (2026-03-12)

### Bug Fixes

* **Editor:** DP-179668 some toolbar buttons not returning focus to editor ([#1117](#1117)) ([67a2a64](67a2a64))
* **Popover:** DP-175009 revert visibility CSS that caused tippy-box click stealing ([#1119](#1119)) ([82ffcb6](82ffcb6))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-visual-test Add this tag when the PR does not need visual testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants