Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
On the Forums page (admin/structure/forums), there is a table with columns NAME and OPERATION. The OPERATION "edit forum" button is too low. To recreate, enable the forum module and go to admin/structure/forums.
Proposed resolution
Modify dropbutton.css to resolve forum operations but still adhere to split button functionality desired in Issue #2282599.
Comment | File | Size | Author |
---|---|---|---|
#3 | Text_formats_and_editors___Drop8.png | 96.52 KB | evilehk |
#3 | interdiff-1-3.txt | 631 bytes | evilehk |
#3 | forum_edit_links_in-2350387-3.patch | 517 bytes | evilehk |
#2 | 2350387_text_formats_spacing.jpg | 63.03 KB | SteffenR |
#2 | 2350387_text_formats.jpg | 63.61 KB | SteffenR |
Comments
Comment #1
evilehk CreditAttribution: evilehk commentedPatch and screenshots of fix, and a screenshot showing #2282599: Dropdown menus in most of core are broken in Stark still working in Stark.
Comment #2
SteffenRPatch works fine for me. I checked the CSS on several administrational pages with only a "simple" edit button in a row. It also solves the problem we have at Text-Formats administration page:
The styling of the dropdown buttons is not affected by the style change.
@evilehk
Maybe we could fix the issue with uneven spacing of buttons and the table rows - but this is a more general issue i saw on lot of administrational pages.
SteffenR
Comment #3
evilehk CreditAttribution: evilehk commentedGreat input, SteffenR, thanks! I modified the title of this issue since the problem does not apply to the forum page only.
I brought back the absolute positioning for the dropbutton, but added a transform to align vertically. I left the dropdown buttons with multiple options alone. Otherwise they would have centered themselves vertically when opened. I tested the admin views page, menu page, forum page, and text formats in Stark and Seven. Attaching patch, interdiff, and screenshot of text format page.
Comment #6
idebr CreditAttribution: idebr commentedThis was fixed in #2349829: #2293589 breaks table row buttons