Webform Bootstrap vs. Radix

Created on 30 July 2024, about 1 year ago

Problem/Motivation

We're using the famous Radix Drupal Theme β†’ and were unsure if we *should* use the Webform Bootstrap Submodule *or not*.

Looking deeper into the code then, we found this line:
https://git.drupalcode.org/project/webform/-/blob/6.2.x/modules/webform_...

So it seems this submodule is only intended for the Bootstrap Theme, not for the Bootstrap ecosystem (which is radix also based on).

So I'd like to suggest adding a clear sentence to the module info file like "Note: This is only to be used with the Drupal Bootstrap Theme, NOT with other Bootstrap Framework related Themes (like Radix)" (at least, until / if it has this strict hidden condition!)

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

πŸ’¬ Support request
Status

Active

Version

6.3

Component

Documentation

Created by

πŸ‡©πŸ‡ͺGermany Anybody Porta Westfalica

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @Anybody
  • πŸ‡©πŸ‡ͺGermany Anybody Porta Westfalica
  • Status changed to Needs review about 1 year ago
  • πŸ‡©πŸ‡ͺGermany Anybody Porta Westfalica

    Please feel free to improve my sentence, as I'm not a native speaker :)

  • πŸ‡©πŸ‡ͺGermany Anybody Porta Westfalica

    PS: Of course we could also discuss extending this to Radix in the future or if Radix is too different and would need a separate submodule then, eventually based on this one.

  • πŸ‡©πŸ‡ͺGermany Anybody Porta Westfalica
  • Pipeline finished with Success
    about 1 year ago
    Total: 2555s
    #238567
  • Status changed to Needs work about 1 year ago
  • Pipeline finished with Success
    9 months ago
    Total: 877s
    #377095
  • Pipeline finished with Failed
    9 months ago
    Total: 903s
    #384113
  • Pipeline finished with Failed
    9 months ago
    Total: 717s
    #388508
  • Pipeline finished with Failed
    9 months ago
    #388966
  • Pipeline finished with Skipped
    9 months ago
    #389189
  • Pipeline finished with Canceled
    7 months ago
    Total: 161s
    #421118
  • Pipeline finished with Success
    7 months ago
    Total: 966s
    #421121
  • Pipeline finished with Skipped
    7 months ago
    #421420
  • Pipeline finished with Failed
    6 months ago
    Total: 213s
    #445735
  • Pipeline finished with Failed
    6 months ago
    Total: 208s
    #445748
  • Pipeline finished with Failed
    6 months ago
    Total: 304s
    #445801
  • Pipeline finished with Failed
    6 months ago
    Total: 214s
    #445833
  • Pipeline finished with Canceled
    6 months ago
    Total: 101s
    #445836
  • Pipeline finished with Failed
    6 months ago
    Total: 214s
    #445838
  • Pipeline finished with Failed
    6 months ago
    Total: 635s
    #445851
  • Pipeline finished with Failed
    6 months ago
    Total: 233s
    #445867
  • Pipeline finished with Success
    6 months ago
    Total: 203s
    #445870
  • Pipeline finished with Success
    6 months ago
    Total: 230s
    #445877
  • Status changed to Fixed 13 days ago
  • πŸ‡ΊπŸ‡ΈUnited States jrockowitz Brooklyn, NY

    I committed this tweak.

  • Now that this issue is closed, please review the contribution record.

    As a contributor, attribute any organization helped you, or if you volunteered your own time.

    Maintainers, please credit people who helped resolve this issue.

    • d27c98d4 committed on 6.3.x
      [#3464731] feat: webform_bootstrap: Bootstrap Theme vs. Bootstrap...
    • d27c98d4 committed on 6.x
      [#3464731] feat: webform_bootstrap: Bootstrap Theme vs. Bootstrap...
Production build 0.71.5 2024