Problem/Motivation

There appears to be no hover style for <details><summary> elements for the Seven theme.
Tested on Firefox, Chrome + Safari, and there's no indication you're hovering over the summary.

Discovered while working on #3102724: Improve usability of core compatibility ranges on available updates report
Closely related to (and should be blocked on) #2980304: Seven theme's details/summary focus style is broken/missing in some browsers.
#2980304: Seven theme's details/summary focus style is broken/missing in some browsers. deals with the fact that text-decoration CSS doesn't reliably work on <summary> on all browsers by adding a <span> inside the <summary> (for which text-decoration works fine).

Proposed resolution

Add appropriate styles.
Probably will just have a weak text-decoration: underline for consistency with everything else in Seven.
Wait for the <span> from #2980304: Seven theme's details/summary focus style is broken/missing in some browsers. so we can get the underline working reliably on all browsers.

CommentFileSizeAuthor
#18 4f851b4d2ac504baf11d05ecb2867aef04093209.png14.84 KBavpaderno

Issue fork seven-3116331

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

dww created an issue. See original summary.

Version: 8.9.x-dev » 9.1.x-dev

Drupal 8.9.0-beta1 was released on March 20, 2020. 8.9.x is the final, long-term support (LTS) minor release of Drupal 8, which means new developments and disruptive changes should now be targeted against the 9.1.x-dev branch. For more information see the Drupal 8 and 9 minor version schedule and the Allowed changes during the Drupal 8 and 9 release cycles.

dww’s picture

dww
we/he/they
commented
Title: [PP-1] Seven theme's details/summary doesn't have a hover style » Seven theme's details/summary doesn't have a hover style

#2980304: Seven theme's details/summary focus style is broken/missing in some browsers. is in, so this can proceed.

Version: 9.1.x-dev » 9.2.x-dev

Drupal 9.1.0-alpha1 will be released the week of October 19, 2020, which means new developments and disruptive changes should now be targeted for the 9.2.x-dev branch. For more information see the Drupal 9 minor version schedule and the Allowed changes during the Drupal 9 release cycle.

Version: 9.2.x-dev » 9.3.x-dev

Drupal 9.2.0-alpha1 will be released the week of May 3, 2021, which means new developments and disruptive changes should now be targeted for the 9.3.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 9.3.x-dev » 9.4.x-dev

Drupal 9.3.0-rc1 was released on November 26, 2021, which means new developments and disruptive changes should now be targeted for the 9.4.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 9.4.x-dev » 9.5.x-dev

Drupal 9.4.0-alpha1 was released on May 6, 2022, which means new developments and disruptive changes should now be targeted for the 9.5.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Project: Drupal core » Seven
Version: 9.5.x-dev » 1.0.0-alpha1
Component: Seven theme » Code

The Seven theme has been removed from Drupal 10 core. I confirmed that this issue only affects Seven and no other themes included with Drupal core, so I am moving this to the contributed Seven project.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Version: 1.0.0-alpha1 » 1.0.x-dev
Issue summary: View changes
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Parent issue: #3102724: Improve usability of core compatibility ranges on available updates report »
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Title: Seven theme's details/summary doesn't have a hover style » Details/summary doesn't have a hover style

avpaderno credited bnjmnm.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Active » Closed (outdated)

I apologize: 10:46 PM is not the right time for me. 😉

The :focus styles are present, but the :hover styles are not.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Assigned: Unassigned » avpaderno
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Title: Seven theme's details/summary doesn't have a hover style » details/summary doesn't have a hover style

avpaderno opened merge request !143

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Assigned: avpaderno » Unassigned
Status: Active » Needs review
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
StatusFileSize
new 4f851b4d2ac504baf11d05ecb2867aef04093209.png14.84 KB

I took this screenshot from the /admin/modules page. Are there other pages which could be used to check?

screenshot

  • avpaderno committed 6dc5458a on 1.0.x 
    Issue #3116331: details/summary doesn't have a hover style

  • avpaderno committed 9ecac7a3 on 2.0.x 
    Issue #3116331: details/summary doesn't have a hover style
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Needs review » Fixed
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Issue summary: View changes
Issue tags: +CSS

Status: Fixed » Closed (fixed)

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