Page MenuHomePhabricator

Link button remains depressed after canceling out of link inspector in insertion mode
Closed, ResolvedPublic8 Story Points

Description

  1. Put cursor on an empty line
  2. Click link button in toolbar
  3. Link button is now depressed (correct)
  4. Click Cancel in link inspector
  5. Link button is still depressed (incorrect)

Event Timeline

Catrope created this task.May 12 2015, 11:44 AM
Catrope raised the priority of this task from to Needs Triage.
Catrope updated the task description. (Show Details)
Catrope added a subscriber: Catrope.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMay 12 2015, 11:44 AM
Jdforrester-WMF triaged this task as Normal priority.May 12 2015, 11:53 AM
Jdforrester-WMF set Security to None.

This is something in VE core or OOjsUI, so removed the VE-MW-Links project. Annotation inspectors have close actions, but I don't think you can properly go back to the tool and ensure it's deactivated from there when the close action is used...

<MatmaRex> first, i note that when pressing Ctrl+K to open the inspector, the toolbar tool doesn't get activated
<Krenair> I feel like I've seen a task for that before
<MatmaRex> i'm not sure if we want it to
<MatmaRex> but assuming that we do, the best way to fix this seems to emit 'updateState' events on the toolbar, and handle them in each tool (to activate the tool is corresponding dialog is open)
<MatmaRex> (and deactivate when closed)
<MatmaRex> this way that tool wouldn't have to have access directly to the dialog
<MatmaRex> emit 'updateState' events on the toolbar when a dialog is opened/closed*

Change 221169 had a related patch set uploaded (by Bartosz Dziewoński):
Specify 'associatedWindows' for link and citation tools

https://gerrit.wikimedia.org/r/221169

Change 221170 merged by jenkins-bot:
Synchronize toolbar buttons state with state of their opened windows

https://gerrit.wikimedia.org/r/221170

Jdforrester-WMF closed this task as Resolved.Jun 29 2015, 11:27 PM
Jdforrester-WMF removed a project: Patch-For-Review.
Jdforrester-WMF edited a custom field.

Change 221169 merged by jenkins-bot:
Specify 'associatedWindows' for link and citation tools

https://gerrit.wikimedia.org/r/221169