Last modified: 2014-10-10 18:07:07 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 T73726, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 71726 - Provide toclimit-2 and .toclevel-1 support in tablet table of contents
Provide toclimit-2 and .toclevel-1 support in tablet table of contents
Status: RESOLVED WONTFIX
Product: MobileFrontend
Classification: Unclassified
Feature requests (Other open bugs)
unspecified
Tablet PC All
: Unprioritized enhancement
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-10-07 02:03 UTC by stillwaterising
Modified: 2014-10-10 18:07 UTC (History)
9 users (show)

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


Attachments

Description stillwaterising 2014-10-07 02:03:46 UTC
Template:TOC limit does not work in mobile browsers that show the TOC by default. 

Example URL: http://en.m.wikipedia.org/w/index.php?title=Islamic_State_of_Iraq_and_the_Levant&oldid=628558101

Expected Results: Only level 2 heading shown (as is done on Desktop view)

Actual Results: Headings are not limited in mobile view
In source view it looks like:
<div class="toclimit-2"></div>
<div class="toc-mobile">
  <h2 aria-controls=....
</div>

Additional Builds and Platforms: Tried multiple mobile user agents on Firefox
Comment 1 Bingle 2014-10-07 02:05:18 UTC
Prioritization and scheduling of this bug is tracked on Trello card https://trello.com/c/5iungbbW
Comment 2 Redrose64 2014-10-07 09:28:08 UTC
I'm pretty sure that it's because the appropriate rule is in [[:en:MediaWiki/Common.css]] but not in [[:en:MediaWiki/Mobile.css]]
Comment 3 Redrose64 2014-10-07 09:29:42 UTC
Whoops. I meant [[:en:MediaWiki:Common.css]] and [[:en:MediaWiki:Mobile.css]].

Will Phabricator offer (a) a "preview" feature and (b) the ability to amend your own posts?
Comment 4 Jon 2014-10-07 21:50:04 UTC
The table of contents in the mobile site is generated totally different from the desktop skin to give us more control to style it/position it appropriately for mobile/tablets. We currently do not support the css selectors that are used in Common.css and I'm not sure if we would want to. Thus marking as an enhancement for further consideration.
Comment 5 Florian 2014-10-08 16:13:56 UTC
I'm not sure what the advantage of this is (or would be). The ToC is the basic element where the user can orient when reading the article, so why remove (maybe important and interesting) contents? Just my opinion :)
Comment 6 Jon 2014-10-10 18:07:07 UTC
After discussing with other MobileFrontend developers we will not be adding classes to support this kind of thing.

If you want to do this on a global basis you can already do 
.toc-mobile ul ul { display: none; }

Doing this on a per template basis would be possible with the completion of https://www.mediawiki.org/wiki/Requests_for_comment/Allow_styling_in_templates
but we won't be doing this as part of the MobileFrontend extension as we cannot guarantee we will maintain it.

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


Navigation
Links