- Issue created by @aryashreep
- Status changed to Needs review
over 1 year ago 8:50am 27 July 2023 - 🇮🇳India ravi kant Jaipur
@aryashreep
This was due todata-toggleattribute changed in Bootstrap5 asdata-bs-toggle
Attaching path. - 🇮🇳India mukesh-kumar3 Dharmshala
The issue lies in Bootstrap 5 where the attribute
"data-toggle=tab"has been replaced with"data-bs-toggle=tab". Also we need to remove thelifrom the twig template. To address this problem, I am implementing a patch that specifically caters to Bootstrap 5. - 🇮🇳India mukesh-kumar3 Dharmshala
My bad i didn't check the #2 patch while adding my patch. In #2 patch there is an issue of multiple active file and #3 i resolve the multiple active issue.
- last update
over 1 year ago Composer error. Unable to continue. - 🇮🇳India aryashreep
Hi @ravi kant & @mukesh-kumar3,
Thank you for the patch. It is working as expected. I have added your patch sent request for the merge.
Thank you.
- Status changed to Needs work
over 1 year ago 6:51pm 31 July 2023 - 🇺🇸United States mmarler
The fix should also support Bootstrap 3 and 4. Both use
'data-toggle' => 'tab'class. Also, in #3 we'll want to keep the- tags.
- Status changed to Needs review
over 1 year ago 11:07am 1 August 2023 - Status changed to Needs work
over 1 year ago 6:24am 2 August 2023 - 🇮🇳India ravi kant Jaipur
@aryashreep
According to me, we can provide a dropdown of Bootstrap versions.
<select><option>Bootstrap-3</option><option>Bootstrap-4</option><option>Bootstrap-5</option></select>
A conditional statement written in module file on based key value of dropdown. I've confirmed that the tabs can have both data-toggle and data-bs-toggle without issue. If there's a way to fix the multiple active tab problem without removing
- , it should work with Bootstrap 3, 4, and 5. Unfortunately I haven't had luck figuring that out.
This patch should work with Bootstrap 3, 4, and 5. It might be too narrowly scoped - it removes the
<li>tag if the name of the theme is "bootstrap5". Not sure if there's a way to make it recognize any theme using Bootstrap 5.- Status changed to Postponed
about 1 year ago 2:49pm 30 October 2023 - 🇺🇸United States mark_fullmer Tucson
This patch should work with Bootstrap 3, 4, and 5. It might be too narrowly scoped - it removes the
- tag if the name of the theme is "bootstrap5". Not sure if there's a way to make it recognize any theme using Bootstrap 5.
Yes, this would be too narrowly scoped as-is. The expectation of this module is that the Bootstrap framework may be implemented in any manner, not just through the Drupal theme "Bootstrap 5."
As with the problem noted in 💬 Vertical tabs not working with bootstrap 3 -- help? RTBC , it appears that the differences between Bootstrap 3, 4, and 5 necessitate some level of conditional logic .
I've therefore created ✨ Configuration option to set which version of Bootstrap is being used Fixed . Once that is completed, we can return to this issue and add conditional logic in the code to vary the rendering as needed to match the version of the Bootstrap framework being used.
Setting status to "Postponed."
Agree with #11 and #14. A conditional tied to a Bootstrap version selector would be a good fix.
Looking at #3 I confirmed that the
data-bs-toggledata-toggleattributes can both be included without issue. However, I've been unable to find any way to get Bootstrap 3 and 4 working without the<li>wrapper, or any way to get Bootstrap 5 working without removing it.- Status changed to Active
11 months ago 2:39pm 31 January 2024 - 🇺🇸United States mmarler
The 2.x branch has a bootstrap version selector to help add conditional logic to this issue. Changing the status to active.
- last update
10 months ago 1 pass - Status changed to Needs review
10 months ago 10:22pm 23 February 2024 - last update
10 months ago 1 pass - 🇮🇳India ravi kant Jaipur
ravi kant → changed the visibility of the branch 2.x to hidden.
- 🇮🇳India ravi kant Jaipur
ravi kant → changed the visibility of the branch 2.x to active.
- 🇺🇸United States mark_fullmer Tucson
I cannot checkout the MR branch.
I think this is because the merge request branch was unconventionally named 2.x, matching the branch name from the origin. You can work around this by running the following:
git remote add bootstrap_horizontal_tabs-3377313 git@git.drupal.org:issue/bootstrap_horizontal_tabs-3377313.git
git fetch bootstrap_horizontal_tabs-3377313
git pull bootstrap_horizontal_tabs-3377313 2.x - Status changed to Needs work
10 months ago 7:47pm 27 February 2024 - 🇺🇸United States mark_fullmer Tucson
In testing this merge request, I realize that the change introduced to the 2.x branch in ✨ Configuration option to set which version of Bootstrap is being used Fixed makes a backwards-compatibility-breaking assumption: it sets the default version for Bootstrap library to "5," where up till now, this module has assumed that Bootstrap 3/4 is installed.
We should not introduce this new assumption in a minor version release of this module (i.e., 2.x branch). We could make the new default for Bootstrap 5 in a new major version branch (3.x). But for now, this needs to be reworked so that new and existing sites, when they update to a 2.x release of this module with this code change, the default value is "4," not "5."
Why this matters:
1. An existing site that has up till now been assuming that this module, out of the box, works with Bootstrap 3 or 4, when updating, will have their integrations failing.
2. For new sites based on distributions which package configurations of Bootstrap Horizontal Tabs, they would also 'not work' out of the box anymore.How this should be refactored:
1. Set the default for new installs to '4' instead of '5' in config/install/bootstrap_horizontal_tabs.settings.yml
2. Add an update hook for existing sites that populates bootstrap_horizontal_tabs.settings.version to '4'
3. Refactor the #default value in src/Form/BaseConfigurationForm.php from '5' to '4'
4. Create a new issue for a new major version branch of bootstrap_horizontal_tabs that sets the version to '5' instead of '4'. - Status changed to Needs review
10 months ago 7:09pm 1 March 2024 - Status changed to RTBC
9 months ago 6:55pm 29 March 2024 - 🇺🇸United States mark_fullmer Tucson
With the latest changes, this functionally checks out and won't adversely affect existing sites!
- Status changed to Fixed
9 months ago 10:07pm 1 April 2024 Automatically closed - issue fixed for 2 weeks with no activity.