Last modified: 2013-10-04 23:57:09 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 T52254, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 50254 - VisualEditor: Can't remove heading without header markup leaking to the rest of the page if it is the first element of the page
VisualEditor: Can't remove heading without header markup leaking to the rest ...
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
Data Model (Other open bugs)
unspecified
All All
: High normal
: VE-deploy-2013-09-26
Assigned To: Ed Sanders
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-06-26 20:03 UTC by Martijn Hoekstra
Modified: 2013-10-04 23:57 UTC (History)
7 users (show)

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


Attachments

Description Martijn Hoekstra 2013-06-26 20:03:51 UTC
if an article starts with 
==Foo==
Bar Baz Fizz Buzz

It is impossible to remove ==Foo== without having the paragraph take on the h2 markup
Comment 1 Chris McKenna 2013-07-22 18:32:43 UTC
As a workaround, it is possible to convert the header to paragraph text and then delete it. That is no better though than deleting and then converting the changed text back to a paragraph.
Comment 2 Chris McKenna 2013-07-22 18:36:25 UTC
Actually, this bug looks to be a subset of Bug 51829 is this best merged or that bug better split?
Comment 3 Martijn Hoekstra 2013-07-22 20:27:06 UTC
They definitely seem related, with the distinction that this but has the problem that there is no slug (am I using the lingo right) on the line above the first header if the header is the first element of the page. Merge however you feel is most useful.
Comment 4 Amir E. Aharoni 2013-07-26 08:17:29 UTC
It happens to me always, even if the heading is in the middle of the page.
Comment 5 Ed Sanders 2013-09-23 14:23:23 UTC
This is a separate issue to bug 51829 (now bug 50100).
Comment 6 Gerrit Notification Bot 2013-09-23 14:23:55 UTC
Change 85673 had a related patch set uploaded by Esanders:
Delete empty nodes instead of merging into them

https://gerrit.wikimedia.org/r/85673
Comment 7 Ed Sanders 2013-09-23 14:26:12 UTC
The delete-when-empty-merge-otherwise behaviour implemented by the patch above is also used by OO/LO & Google docs.
Comment 8 Gerrit Notification Bot 2013-09-24 18:28:52 UTC
Change 85673 merged by jenkins-bot:
Delete empty nodes instead of merging into them

https://gerrit.wikimedia.org/r/85673
Comment 9 Wikifram@gmail.com (Account disabled) 2013-10-04 06:44:31 UTC
Nice that this is solved, except that it isn't. I had already noted that at MediaWiki, it still blanks the header (but doesn't add nowiki). Tested today at enwiki, and sadly, even worse, again a nowiki was added inside the header instead of removing the empty header: [https://en.wikipedia.org/w/index.php?title=Robbers_Cave_State_Park&diff=575691345&oldid=558376701]

Has this patch ever worked anywhere? Diff?
Comment 10 Ed Sanders 2013-10-04 08:49:37 UTC
You're confusing this with bug 50100.
Comment 11 Wikifram@gmail.com (Account disabled) 2013-10-04 09:19:21 UTC
(In reply to comment #10)
> You're confusing this with bug 50100.

Not my mistake though, this was reported like this in the VisualEditor weekly update - 2013-09-26 (MW 1.22wmf19) by Jdforrester:

"Blanking the contents of a heading, pre-formatted block or other formatting block now deletes the block rather than leaving it empty, which is consistent with how OpenOffice and Google Docs behave (bug 50254)." 

I noted that what the VE status report describes is ''not'' fixed, and followed the link number given in the VE status report. If this bug is about something else than what the VE status report described, then this is just one more in the already long list of things wrong with that status report (for which it is apparently "impossible" to write an update or correction in any way, shape or form). 

Anyway, you still get the behaviour as described: Go to the first word you want to keep (after the initial header), and use backspace to remove the header: your next paragraph now becomes the header. Only when you first empty the header (leaving the line beneath it), and then go to the text and use backspace, is it possible to remove a starting header without converting the following text into a header. I wouldn't call this "solved", perhaps "somewhat improved" assuming that the latter possibility / workaround didn't work in the past. (I note that you only go to the start of the heading, and then select "downwards" instead of "sideways"; deleting the header like this works as well)

Note also that if yu eventually get rid of that header, it seems to be impossible to remove a first empty line from an article: [https://en.wikipedia.org/w/index.php?title=User%3AFram%2Fsandbox&diff=575703425&oldid=575703070]. Probably needs a different bug (or already exists), but not really optimal either.
Comment 12 Ed Sanders 2013-10-04 09:43:39 UTC
Steady on, they're similar bugs and easy to confuse.

The case you describe at the end does sound like a separate bug.
Comment 13 Ed Sanders 2013-10-04 09:49:12 UTC
Filed as bug 54964
Comment 14 Wikifram@gmail.com (Account disabled) 2013-10-04 09:51:46 UTC
(ec) Yeah, I'm a bit frustrated by the number of errors in that status update, and the apparent lack of possibilities to correct these (even as an addendum or some such) and lack of care about this among the people responsible for this. Not your fault though. Thanks for filing that bug.

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


Navigation
Links