Problem/Motivation

Currently the JS code is hard-coded to look for the form element classes provided by Claro, the default Drupal admin theme. Those classes depend on the theme enabled for each form, so make them configurable for non-Claro themes.

Issue fork menu_parent_form_ui-3552420

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

jastraat created an issue. See original summary.

jastraat opened merge request !3

jastraat’s picture

jastraat commented
Status: Active » Needs review
jasongose’s picture

jasongose commented
Status: Needs review » Reviewed & tested by the community

There's a bit of spacing that should be adjusted but otherwise this looks good to me :)
(and just so we're clear, I'm happy to give it another look if you adjust the spacing)

jastraat’s picture

jastraat commented

Thanks Jason! I'll create a followup issue to clean up the extractor JS in more detail. (It needs love.)

  • jastraat committed 773f14e8 on 1.0.x 
    [#3552420] feat: Make menu element wrapper classes configurable to...
jastraat’s picture

jastraat commented
Status: Reviewed & tested by the community » Fixed

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

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

Maintainers, please credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.