Last modified: 2014-06-09 16:44:26 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T60371, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 58371 - VisualEditor: "Clear formatting" should never be highlighted
VisualEditor: "Clear formatting" should never be highlighted
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
Editing Tools (Other open bugs)
unspecified
All All
: Low normal
: VE-deploy-2014-01-02
Assigned To: Ed Sanders
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-12-12 04:10 UTC by Krinkle
Modified: 2014-06-09 16:44 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Selecting text spanning across a link shows "Clear formatting" active (44.17 KB, image/png)
2013-12-12 04:12 UTC, Krinkle
Details
When using the link inspector, "Link" and "Clear formatting" are active (71.22 KB, image/png)
2013-12-12 04:13 UTC, Krinkle
Details

Description Krinkle 2013-12-12 04:10:28 UTC
I'm not sure if this was done on purpose, but when selecting text containing some formatting in some or all of the selected text (e.g. part of a link and some text after it, or part of a bolded string) the toolbar shows the "Clear formatting" button highlighted (similarly to how the "Bold" icon is highlighted when selecting bolded text).

The toolbar is supposed to reflect current state of selected text. Showing the "Clear formatting" button already activated seems like an impossible state (it would theoretically mean some hypothetical "clear" annotation is applied to the selected text, which is incorrect. The only case in which that would technically be correct is if you apply text that is already cleared of formatting).

This highlighting for the "Clear formatting" button is confusing from a UX point of view. I would associate it with how "bold" is highlighted when bolding is applied.

When encountering this strange UX behaviour earlier today, it took me a while to figure out what it is supposed to mean (longer than it should for an intuitive editor). I don't think this is because I'm familiar with VisualEditor's previous state, but more in general.

I think the correct approach would be to never highlight the clear formatting button in the toolbar, and have it be an "apply" button, you select text, and then activate (not deactivate) "clear formatting".

This concept also exists in other popular editors.

Alternatively, though that would be a new concept for people to learn, perhaps change the icon to something that means the opposite of no formatting, so that it being turned on for selected text, and being able deactivate it, like it currently does, actually makes sense as an action take.
Comment 1 Krinkle 2013-12-12 04:12:11 UTC
Created attachment 14068 [details]
Selecting text spanning across a link shows "Clear formatting" active
Comment 2 Krinkle 2013-12-12 04:13:16 UTC
Created attachment 14069 [details]
When using the link inspector, "Link" and "Clear formatting" are active
Comment 3 James Forrester 2014-01-18 06:06:59 UTC
This was done in the changes to the toolbar.
Comment 4 Roan Kattouw 2014-03-20 00:12:19 UTC
This now happens again. If you:

1. Select an image
2. Click Bold. This bolds the caption (this is a separate bug)
3. Select something else, then select the image again (otherwise the clear formatting button doesn't get enabled)
3. Click Clear Formatting.
4. Observe that Clear Formatting is now depressed.

Obviously there are lots of other weird things going on, but it seems that the only way that Clear Formatting button manages to not be depressed is by assuming that it will always successfully clear all formatting and therefore be disabled next.
Comment 5 James Forrester 2014-06-09 16:44:26 UTC
That bit is now fixed too AFAICT.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links