GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Edit Task
  • Edit Related Tasks...
  • Create Subtask
  • Edit Parent Tasks
  • Edit Subtasks
  • Merge Duplicates In
  • Close As Duplicate
  • Edit Commits
  • Edit Mocks
  • Mute Notifications
  • Protect as security issue
  • MediaWiki-Parser
  • (Backlog)

This view

  • Original page
  • New address

GeezerVison is showing phabricator.wikimedia.org — T28375 Italics should be passed to TOC

Page MenuHomePhabricator
Log In
Create Task
Maniphest T28375

Italics should be passed to TOC
Closed, ResolvedPublic
Actions

  • Edit Task
  • Edit Related Tasks...
  • Create Subtask
  • Edit Parent Tasks
  • Edit Subtasks
  • Merge Duplicates In
  • Close As Duplicate
  • Edit Related Objects...
  • Edit Commits
  • Edit Mocks
  • Mute Notifications
  • Protect as security issue
Assigned To
None
Authored By
• bzimport
Dec 20 2010, 3:43 PM2010-12-20 15:43:00 (UTC+0)
Project Tags
  • MediaWiki-Parser (Backlog)
Referenced Files
F6981: italics.patch
Nov 21 2014, 11:12 PM2014-11-21 23:12:08 (UTC+0)
Subscribers
DanielFriesen
hashar
Platonides
• wikibugs-l-list

Description

Author: gdonato

Description:
Submitting on behalf of those at WP:VPT.

Fairly simple report, italics should be represented on the table of contents in the same way as <sup> etc. in bug 8393. Perhaps other things should also be added e.g. bold? I don't know...

Version: unspecified
Severity: enhancement
URL: http://en.wikipedia.org/w/index.php?title=Wikipedia:Village_pump_%28technical%29&oldid=403630551#Italics_in_ToC

Details

Reference
bz26375

Event Timeline

• bzimport raised the priority of this task from to Medium.Nov 21 2014, 11:12 PM2014-11-21 23:12:08 (UTC+0)
• bzimport added a project: MediaWiki-Parser.
• bzimport set Reference to bz26375.
• bzimport added a subscriber: Unknown Object (MLST).
• bzimport created this task.Dec 20 2010, 3:43 PM2010-12-20 15:43:00 (UTC+0)
• bzimport added a comment.Dec 20 2010, 5:05 PM2010-12-20 17:05:18 (UTC+0)
Comment Actions

gdonato wrote:

v. simple patch

Very simple, adds <i>, <em> to regex. Probably insufficient.

Attached:

italics.patch736 BDownload

• bzimport added a comment.Nov 6 2011, 2:38 PM2011-11-06 14:38:38 (UTC+0)
Comment Actions

sumanah wrote:

gdonato, I am sorry that it took so very long for you to get a response to your patch. Thank you for your contribution. We are aiming at being much more responsive now and in the future.

Unfortunately, your patch now does not cleanly apply to trunk, since there have been so many changes since December of last year. Would you be interested in updating it?

Also, you may be interested in joining the parser/visual editor mailing list to see what changes are happening in the parser: https://lists.wikimedia.org/mailman/listinfo/wikitext-l

Thanks, and again, my regrets.

Platonides added a comment.Nov 6 2011, 3:10 PM2011-11-06 15:10:52 (UTC+0)
Comment Actions

It wasn't that outdated. The line was there, it was moved 203 lines below.
Fixed in r102179. I only whitelisted <i> since that's what we are generating from ''.
Also added parser test.

hashar added a comment.Dec 6 2011, 10:51 AM2011-12-06 10:51:02 (UTC+0)
Comment Actions

Bold enabled as a followup with r105284

Log In to Comment
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.