Last modified: 2014-08-11 17:42:24 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 T70101, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 68101 - VisualEditor: Don't allow a linking action if no text was highlighted before
VisualEditor: Don't allow a linking action if no text was highlighted before
Status: RESOLVED WONTFIX
Product: VisualEditor
Classification: Unclassified
General (Other open bugs)
unspecified
All All
: Unprioritized normal
: ---
Assigned To: Editing team bugs – take if you're interested!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-07-16 13:39 UTC by Elitre
Modified: 2014-08-11 17:42 UTC (History)
5 users (show)

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


Attachments

Description Elitre 2014-07-16 13:39:18 UTC
Load VE. Say "Foobar" is the first word on the page.
If you click on the link tool, by mistake or not, without selecting anything before, Foobar will automatically become a link to the Foobar page (it doesn't help that pressing ESC doesn't abort the action, the link gets applied anyway, as we know).
NicoV noticed this in several French articles, where you can quite tell links happened "by mistake" since the first word is an article, "le".
Comment 1 James Forrester 2014-07-16 17:59:12 UTC
This is explicitly a feature that was requested (years ago now). People clicking buttons and not paying attention to what they're doing seems more like a social issue than a software bug.
Comment 2 NicoV 2014-07-17 05:30:17 UTC
I was the one reporting the issue with words like "le" being made into internal links by mistake on frwiki. I think the behavior of automatically selecting the entire word when make an internal link should be kept (it's probably causing less issues than it's avoiding).

But the main problem with this is that the ESC key is not working correctly (it should cancel the action and not silently validate it). Do you plan to do something to make the ESC key work as it works in every other application ?
Comment 3 NicoV 2014-08-10 17:35:20 UTC
Please read my comment: the main problem is ESC key not working at all as could be expected by any user.
Comment 4 James Forrester 2014-08-10 18:58:52 UTC
(In reply to NicoV from comment #2)
> But the main problem with this is that the ESC key is not working correctly
> (it should cancel the action and not silently validate it). Do you plan to
> do something to make the ESC key work as it works in every other application
> ?

You are mistaken. The Escape key is doing exactly what users expect; it closes the window. The user clicking the link button – not the closing the link editor window – is what makes the link.
Comment 5 NicoV 2014-08-10 19:04:49 UTC
Well, give me one example of a software behaving like that: in any software I know, when you click on a button that open a dialog, if you cancel the dialog then no action is done.

The current behavior is probably the reason of many links created in frwiki to pages "Le", "La", "Les", ... (all created by VE) which we have to fix on a regular basis. And we obviously miss all the other links created by mistake.

Please, change the current behavior, so that Esc key behaves as in any normal software.
Comment 6 James Forrester 2014-08-10 19:35:57 UTC
Please don't edit war on Bugzilla. Create a new bug about an actual issue if you wish.
Comment 7 Arthur Rubin 2014-08-11 16:00:21 UTC
I don't know how to reopen this.  As an Admin on en.Wikipedia, I can say that having "Escape" _not_ close the box and take none of the actions indicated in the box is a deal-stopper for me.  I will argue against VE be made default if this isn't fixed.
Comment 8 Andre Klapper 2014-08-11 16:48:31 UTC
(In reply to Arthur Rubin from comment #7)
> I don't know how to reopen this.

You were asked to not reopen this, but to file a new and separate enhancement request for the Esc behavior if you wish so.
Comment 9 Arthur Rubin 2014-08-11 17:02:21 UTC
(In reply to Andre Klapper from comment #8)
> (In reply to Arthur Rubin from comment #7)
> > I don't know how to reopen this.
> 
> You were asked to not reopen this, but to file a new and separate
> enhancement request for the Esc behavior if you wish so.

So, you want me to lie and call it an "enhancement request" rather than a bug fix.  _That_, I won't do.  I'll still call it a bug fix, and apparently a serious one.
Comment 10 Andre Klapper 2014-08-11 17:40:42 UTC
I hereby distance myself from that guy who wrote "enhancement" in comment 8, as differing interpretations and judgements of that term might trigger "lies". Please feel free to argument why not supporting Esc is a "serious bug" in a separate, non-enhancement request if you wish so. Thank you.
Comment 11 Andre Klapper 2014-08-11 17:42:24 UTC
Ah. Bug 69407. Alright, thanks.

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


Navigation
Links