? Success

User tests: Successful: Unsuccessful:

avatar Mathewlenning
Mathewlenning
28 Sep 2015

Example of how pull request #7948 can simplify our lives.

This is less than ideal. I'd actually prefer to just send the wrapper and let the developer add the tab panes using the JHtmlElement interface.

But even in its current form it is way better than the current JHtmlTabs implementation.

BTW does anyone know why we are adding the empty dt/dd set? or can I remove it?

10408cb 28 Sep 2015 avatar Mathewlenning Doh!
avatar Mathewlenning Mathewlenning - open - 28 Sep 2015
avatar Mathewlenning Mathewlenning - change - 28 Sep 2015
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Sep 2015
Labels Added: ?
avatar zero-24 zero-24 - change - 28 Sep 2015
Category Libraries
avatar dgt41
dgt41 - comment - 28 Sep 2015

@Mathewlenning I think you based this on the old html/tabs.php which is not the bootstrap version, thus the dd, dl elements and also the Mootools library dependency and the extra tabs.js script. The bootstrap version should be located inside the html/bootstrap.php file

avatar Mathewlenning
Mathewlenning - comment - 28 Sep 2015

O.k. Thanks @dgt41

I'm glad to know that JHtmlTabs isn't actually being used! I'll make a different example after I've looked at the bootstrap file.

avatar Mathewlenning Mathewlenning - close - 28 Sep 2015
avatar Mathewlenning Mathewlenning - change - 28 Sep 2015
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2015-09-28 12:35:42
Closed_By Mathewlenning
avatar Mathewlenning Mathewlenning - close - 28 Sep 2015
avatar Mathewlenning Mathewlenning - head_ref_deleted - 28 Sep 2015

Add a Comment

Login with GitHub to post a comment