Last modified: 2013-11-25 10:28: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 T49139, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 47139 - jQuery.makeCollapsible: support table captions
jQuery.makeCollapsible: support table captions
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
JavaScript (Other open bugs)
unspecified
All All
: Low enhancement with 1 vote (vote)
: 1.23.0 release
Assigned To: Theopolisme
gci2013 https://www.mediawiki.org/wik...
: easy
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-04-11 21:15 UTC by Michael Zajac
Modified: 2013-11-25 10:28 UTC (History)
6 users (show)

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


Attachments

Description Michael Zajac 2013-04-11 21:15:29 UTC
1. When a table is collapsed, the caption remains visible. It should be hidden too, as part of the table’s content. (I’m referring to the table’s <caption> element, not the first table row where the collapse button resides.)

2. If the customToggle is placed within the table caption, it fails to work. It should be made to work in this case, and in this case the table should collapse and leave the caption visible instead of the first row.
Comment 1 Andre Klapper 2013-04-12 10:06:18 UTC
Thanks for taking the time to report this!
Is there any testcase available for this?
Comment 2 Michael Zajac 2013-04-15 20:19:40 UTC
Test cases:

http://en.wiktionary.org/wiki/User:Mzajac/jQuery.makeCollapsible_in_table_captions

Feel free to edit the page to add more tests, correct my errors, or whatever.
Comment 3 Bartosz Dziewoński 2013-05-12 21:20:36 UTC
This should be easy enough; +easy.
Comment 4 Gerrit Notification Bot 2013-11-21 15:41:46 UTC
Change 96773 had a related patch set uploaded by Areke:
jQuery.makeCollapsible: Allow support for table captions

https://gerrit.wikimedia.org/r/96773
Comment 5 Gerrit Notification Bot 2013-11-23 13:21:22 UTC
Change 96773 abandoned by Bartosz Dziewoński:
jQuery.makeCollapsible: Allow support for table captions

Reason:
The GCI task [1] was unclaimed. If you want to continue working on this, click the "Restore Change" button.

[1] https://www.google-melange.com/gci/task/view/google/gci2013/5808344656773120

https://gerrit.wikimedia.org/r/96773
Comment 6 Gerrit Notification Bot 2013-11-23 22:08:18 UTC
Change 97348 had a related patch set uploaded by Theopolisme:
jquery.makeCollapsible: collapse to caption support

https://gerrit.wikimedia.org/r/97348
Comment 7 Theopolisme 2013-11-23 22:08:47 UTC
@Michael, I've written (/am working on) a patch for this that follows the following logic -- could you let me know if it seems okay?

If the table has a caption, collapse to the caption, rather than the first row.

__ Expanded state __

Caption [collapse]
Text         Text
More text    Text and stuff

__ Collapsed state __

Caption [expand]
Comment 8 Michael Zajac 2013-11-24 05:13:22 UTC
That looks great!
Comment 9 Gerrit Notification Bot 2013-11-24 23:07:34 UTC
Change 97348 merged by jenkins-bot:
jquery.makeCollapsible: collapse to caption support

https://gerrit.wikimedia.org/r/97348
Comment 10 Bartosz Dziewoński 2013-11-24 23:10:18 UTC
Done. Thanks!

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


Navigation
Links