Unverified Commit 4569c101 authored by Lauri Timmanee's avatar Lauri Timmanee
Browse files

Issue #2980304 by dww, jrockowitz, Rithesh BK, Kristen Pol, andrewmacpherson,... 

Issue #2980304 by dww, jrockowitz, Rithesh BK, Kristen Pol, andrewmacpherson, dorianwinterfeld, bnjmnm: Seven theme's details/summary focus style is broken/missing in some browsers
parent 2718a80a

core/themes/seven/css/base/elements.css

+10 −0
Original line number Diff line number Diff line
@@ -164,6 +164,16 @@ details summary:focus { 
  text-decoration: underline;

  outline: none;

}

/**

 * Unfortunately, text-decoration for details summary is not supported on all

 * browsers. So we add a span (which can handle text-decoration) in Seven's

 * templates/details.html.twig. In case there are other details templates that

 * don't have the span, we leave the text-decoration in the parent selector.

 * This provides maximum compatibility and coverage with minimal disruption.

 */

details summary:focus span {

  text-decoration: underline;

}

img {

  max-width: 100%;

  height: auto;

core/themes/seven/templates/details.html.twig

+1 −1
Original line number Diff line number Diff line
@@ -30,7 +30,7 @@ 
      ]

    %}

    <summary{{ summary_attributes.addClass(summary_classes) }}>

      {{- title -}}

      <span>{{ title }}</span>

    </summary>

  {%- endif -%}

  <div class="seven-details__wrapper details-wrapper">