Last modified: 2014-11-03 23:56:38 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 T74857, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 72857 - Empty cells displayed in (infobox) table
Empty cells displayed in (infobox) table
Status: RESOLVED INVALID
Product: MobileFrontend
Classification: Unclassified
General/Unknown (Other open bugs)
unspecified
All All
: Unprioritized normal
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks: 64577
  Show dependency treegraph
 
Reported: 2014-11-01 08:10 UTC by Nemo
Modified: 2014-11-03 23:56 UTC (History)
6 users (show)

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


Attachments
Messed up area (15.13 KB, image/png)
2014-11-01 08:10 UTC, Nemo
Details

Description Nemo 2014-11-01 08:10:35 UTC
Created attachment 16990 [details]
Messed up area

http://it.m.wikipedia.org/wiki/Chandigarh_ITF_Juniors_2009_-_Doppio_ragazzi shows the infobox table incorrectly: there are some empty cells added by the mobile site. This is usually something the MediaWiki parser or tidy fix.

Parsoid handles it correctly: http://parsoid-lb.eqiad.wikimedia.org/itwiki/Chandigarh_ITF_Juniors_2009_-_Doppio_ragazzi?oldid=63927067
Comment 1 Bingle 2014-11-01 08:15:21 UTC
Prioritization and scheduling of this bug is tracked on Trello card https://trello.com/c/eALhrZdH
Comment 2 Florian 2014-11-01 15:42:21 UTC
These empty cells aren't added by MobileFrontend, they appear in the desktop website, too (so, the source code output is the same for mobile, desktop and parsoid (they appear in parsoid, too)). The problem seems to be, that MobileFrontend has the following css rule.
.content table td, .content table th {
padding: 3px;
border: 1px #CCC solid;
}

which adds a border around all table cells. I don't know, why the border is added, but i think for better readability :/

Is it possiblle to remove the empty table cells from the template?
Comment 3 Nemo 2014-11-01 16:10:16 UTC
(In reply to Florian from comment #2)
> Is it possiblle to remove the empty table cells from the template?

Last time I asked, it didn't seem possible.
Comment 4 Jon 2014-11-03 18:25:58 UTC
Please add .content table.sinottico td to your MediaWiki:Mobile.css to remove this. We can't do anything our end, it serves the same HTML as desktop the skin is just styling it differently and you'll need to override it to meet your needs. Italian Wikipedia may also want to copy those styles relating to .sinottico [1] to Mobile.css

[1] https://it.wikipedia.org/wiki/MediaWiki:Common.css
Comment 5 Nemo 2014-11-03 22:55:58 UTC
I'm not sure all our table-based templates use that class. Should we override the mobile classes in all our templates?
Comment 6 Jon 2014-11-03 23:56:38 UTC
What other tables have the same issue? If it's all tables you could of course create a generic rule for all tables and simply do 
.content table td { border: none; }

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


Navigation
Links