Problem/Motivation

Steps to reproduce

Proposed resolution

Use BS utility classes: https://getbootstrap.com/docs/5.3/utilities/flex/#justify-content

Not sure how to handle older drowl_base theme versions (Foundation). => They need to override the templates, like mentioned on the module page.

Remaining tasks

User interface changes

API changes

Data model changes

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:

Comments

thomas.frobieter created an issue. See original summary.

thomas.frobieter’s picture

Issue summary: View changes
thomas.frobieter’s picture

Issue summary: View changes
thomas.frobieter’s picture

Not sure how to handle older drowl_base theme versions (Foundation).

As mentioned in the module description, we use Bootstrap classes - old (Foundation based) themes will need to override the affected media templates.

The media templates in 4.x were switched from Foundation to Bootstrap (#3436181: Replace Foundation Twig Templates with Bootstrap Templates) but 4.x will generally also allow to be used with Foundation by overriding and updating the templates.

thomas.frobieter’s picture

@anybody: We hide overlay fields, when "no overlay" is selected (States API). Could you please remove this? It doesnt make sense, since we can disable the overlay for specific breakpoints.

The trigger field is: field_overlay_position, please fix it right here in the issue fork so we dont run into merge conflict ..things.

thomas.frobieter’s picture

Issue summary: View changes
thomas.frobieter’s picture

@anybody: Okay.. there are still fields that should be hidden by States API, so more explicit.. never hide those three:

- field_overlay_display
- field_overlay_button_color
- field_overlay_button_style

While:
- field-overlay-sizing should be hidden when field-overlay-position is set to "disabled" (or unset) => this is already the case
- field_overlay_sizing_md should be hidden when field_overlay_position_md is set to "disabled" (or unset)
- field_overlay_sizing_lg should be hidden when field_overlay_position_lg is set to "disabled" (or unset)

thomas.frobieter’s picture

anybody’s picture

Assigned: thomas.frobieter » anybody
Status: Active » Needs work
thomas.frobieter’s picture

The field states are managed in the wrong module (drowl_header_slides), it doesn't matter if the media slide is a "header slide", a hero or a regular slide.

I'll move it over, so we can finish and release this.

thomas.frobieter’s picture

Status: Needs work » Needs review

Done. Please review @anybody, I'll create a new release afterwards.

I also create a new release of drowl_header_slides without the media slide form fields states stuff.

I'll create another task for the required configuration update, which is not that important.

anybody’s picture

Assigned: anybody » thomas.frobieter
Status: Needs review » Needs work

Looks good, the following things should be done before merging this:
1. Use the empty value instead of "disabled" string for the disabled option
2. Add the configuration updates here, or it will never happen, which will screw up existing projects

I didn't test it myself, just had a look at the states API code.

thomas.frobieter’s picture

Use the empty value instead of "disabled" string for the disabled option

See #3497194: Make media-slide overlay responsive settings mandatory and provide default values, so we dont have an empty value anymore after this change.

Add the configuration updates here, or it will never happen, which will screw up existing projects

I don't think we have any configuration updates.

anybody’s picture

Assigned: thomas.frobieter » Unassigned
Status: Needs work » Reviewed & tested by the community

Ok then I got that wrong. If it works fine for you. it's fine for me!

thomas.frobieter’s picture

Status: Reviewed & tested by the community » Fixed

Merged! thx!

Status: Fixed » Closed (fixed)

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