Last modified: 2014-07-11 21:31:33 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 T49790, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 47790 - VisualEditor: Slugs should be more obvious to the user that they're not "really" blank lines
VisualEditor: Slugs should be more obvious to the user that they're not "real...
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
ContentEditable (Other open bugs)
unspecified
All All
: Normal normal
: VE-deploy-2014-03-06
Assigned To: Ed Sanders
https://de.wikipedia.org/wiki/Benutze...
:
: 50295 52172 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-04-28 16:27 UTC by Raimond Spekking
Modified: 2014-07-11 21:31 UTC (History)
15 users (show)

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


Attachments
Copy of http://i.imgur.com/39QQi8U.png (96.31 KB, image/png)
2013-07-27 21:25 UTC, MZMcBride
Details

Description Raimond Spekking 2013-04-28 16:27:41 UTC
Simple test case, see URL:

* foo
: bar

adds a newline in edit mode. When removing this newline in edit mode the : is killed by the parser.
Comment 1 James Forrester 2013-04-28 17:26:38 UTC
This is deliberate - the line is a "slug", allowing the user to insert text between the UL and the DL without creating a bullet and then down-converting it into a paragraph. We've been thinking about putting a little icon in slugs and making them a different height/etc. so it's obvious that they're not really "there" and are just places items can go.

Re-purposing this bug to talk about that. :-)
Comment 2 James Forrester 2013-07-05 17:55:15 UTC
*** Bug 50797 has been marked as a duplicate of this bug. ***
Comment 3 James Forrester 2013-07-15 02:21:07 UTC
*** Bug 50295 has been marked as a duplicate of this bug. ***
Comment 4 Oliver Keyes 2013-07-15 09:18:43 UTC
Does this include the line of whitespace right at the start of articles?
Comment 5 Raimond Spekking 2013-07-15 17:38:02 UTC
It looks like this bug was fixed somehow. I cannot longer reproduce it on dewiki.
Comment 6 James Forrester 2013-07-15 18:36:06 UTC
(In reply to comment #5)
> It looks like this bug was fixed somehow. I cannot longer reproduce it on
> dewiki.

Yes, we removed the slugs between lists because we felt they were no longer needed given the new editing tools, but they remain more widely.
Comment 7 James Forrester 2013-07-15 18:36:17 UTC
(In reply to comment #4)
> Does this include the line of whitespace right at the start of articles?

Yes.
Comment 8 Oliver Keyes 2013-07-15 18:39:26 UTC
Gotcha. A user was confused as to why they couldn't add a template to the top of an article without it appearing on the same line :).
Comment 9 Joe Decker 2013-07-20 23:11:23 UTC
I suspect this is behind a few of the reports (on the ENWIKI feedback page, some of them mine) in which unexpected deletions are caused.  

As a separate question, can you undo deletion of slugs?  If not, I think that would explain another set of problems others and I have observed.
Comment 10 JohnCD67 2013-07-21 08:55:45 UTC
This causes trouble in the fairly common operation of removing a template (e.g. a PROD) from the top of a page which has an infobox. Click the template to select it, press "Delete" and it goes, leaving apparently a spare blank line at the top. The natural reaction is press "Delete" again to remove the blank line, and that takes out the info box. (Win7, FF22.0)
Comment 11 Ed Sanders 2013-07-27 12:09:17 UTC
Suggestion:
http://i.imgur.com/39QQi8U.png
Comment 12 MZMcBride 2013-07-27 21:25:19 UTC
Created attachment 12988 [details]
Copy of http://i.imgur.com/39QQi8U.png

Please upload attachments to Bugzilla. imgur periodically prunes images; Bugzilla does not. :-)
Comment 13 John Mark Vandenberg 2013-07-28 21:44:51 UTC
*** Bug 52172 has been marked as a duplicate of this bug. ***
Comment 14 Elitre 2013-09-12 16:52:56 UTC
Today JohnCD asked again about this and I got another report from fr.wp, https://fr.wikipedia.org/w/index.php?title=Osmium&diff=96605232&oldid=93301425 .
Hoping this gets some love soon :)
Comment 15 Erik Moeller 2013-10-05 06:11:46 UTC
I agree with previous commenters that this is likely to be at the cause of a lot of the accidental infobox deletions and such. I understand it's a tricky issue, but the current behavior is extremely confusing from the user's point of view, so we need to look at some viable solutions. 

Recapping the simple case: User is on what they perceive to be an unnecessary newline, presses delete to make the visual layout match the expected layout. A separate object (in the case of the infobox, right-aligned on the other side of the screen) suddenly disappears. This is very surprising and counter-intuitive behavior.

Why don't we just suppress the delete key when pressing it would delete the object the slug is associated with? In some alternative editors like Google Docs, it is actually not possible to nuke the entire document by just pressing down the delete key at the top of the page -- it will stop before certain objects. I've never had any problems as a result.

That seems like the simplest solution to me and does not require a redesign of the slugs.
Comment 16 Ed Sanders 2013-10-05 09:02:27 UTC
I think we should split this bug in two: Slugs, and accidental deletion of FocusableNodes. Here is the latter: bug 55336
Comment 17 Jonathan Haas 2013-10-05 09:14:37 UTC
(In reply to comment #15)
> Why don't we just suppress the delete key when pressing it would delete the
> object the slug is associated with?

This would still leave the user with a superfluous empty line, which might be confusing and/or destroying the page layout.

I like the idea in attachment 12988 [details], but I think it would be better, if slugs would have had an effective default height of 0 (for example height: 4px, margin-top: -2px, margin-bottom:-2px), so the article layout would match the real one.
Comment 18 Erik Moeller 2013-10-05 21:24:32 UTC
Thanks, Ed, for splitting the issue and working on a patch for bug 55336. 

Accordingly, I've set that bug to High/Major and this one to Normal, as the overall UX for slugs is important, but not as significant as the accidental deletion of content with the current implementation. I also suspect that we would want to alter the delete behavior even if we end up altering the slug appearance overall.

I do agree that the accidental whitespace is confusing (it's often the most immediate and obvious difference from read mode) and it would be good to find an alternative. In addition to offering an affordance on hover, we might also expand the slugs as the user moves her cursor through the page.
Comment 19 Gerrit Notification Bot 2014-01-24 16:39:10 UTC
Change 109307 had a related patch set uploaded by Esanders:
Collapse block slugs, and expand on hover/focus

https://gerrit.wikimedia.org/r/109307
Comment 20 Gerrit Notification Bot 2014-03-04 14:09:27 UTC
Change 116742 had a related patch set uploaded by Esanders:
Vector style tweaks for collapsible slugs

https://gerrit.wikimedia.org/r/116742
Comment 21 Gerrit Notification Bot 2014-03-05 17:21:02 UTC
Change 109307 merged by jenkins-bot:
Collapse block slugs and expand on hover/focus

https://gerrit.wikimedia.org/r/109307
Comment 22 Gerrit Notification Bot 2014-03-05 18:08:15 UTC
Change 116742 abandoned by Esanders:
Vector style tweaks for collapsible slugs

https://gerrit.wikimedia.org/r/116742
Comment 23 James Forrester 2014-03-06 03:42:54 UTC
Initial version of this now merged and being deployed tomorrow. I think we'll want to revisit the styling and behaviour, but this will do for now.
Comment 24 Ad Huikeshoven 2014-07-11 21:31:15 UTC
On nl.wp a discussion has started to turn on VE. In the process feedback has been collected on VE. Some users commented on "white lines" as a blocking issue for turning VE on as default. 
On top of pages like:
* https://nl.wikipedia.org/wiki/Bali_%28eiland%29?veaction=edit
* https://nl.wikipedia.org/wiki/Justine_Henin?veaction=edit
* https://nl.wikipedia.org/wiki/Rijn?veaction=edit
* https://nl.wikipedia.org/wiki/Brussels_Hoofdstedelijk_Gewest?veaction=edit
* https://nl.wikipedia.org/wiki/Antwerpen_%28stad%29?veaction=edit
as well as on other places on the page "white lines" or carriage returns appear in VE edit mode which do not appear in read mode. On mouse over the white line turns blue. 

Some users fear that newcomers others will delete those white lines and carriage returns. However, after deletion of those lines images are accidently deleted from the page.

This discussion started on bug 49806. James Forrester pointed me to this bug to continue the discussion.

(Deleting white space at the bottom of a page might also delete accidently categories and other metadata.)

How can I help to resolve this 'bug'? I will also copy to bug 55336.

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


Navigation
Links