Problem:
before-patch-firefox.png

Fixes:
firefox-after-patch.png

IE8-after-patch.png

(I guess there was old IEs problem, so that use position: absolute to fix it.)

CommentFileSizeAuthor
#11 fieldset-css-1785978-11.patch2.07 KBmadhavvyas
#8 Account_settings___Site-Install.png258.63 KBjoelpittet
#5 align-problem.jpg93.06 KBdroplet
fieldset-css.patch2.97 KBdroplet
IE8-after-patch.png23.15 KBdroplet
firefox-after-patch.png41.76 KBdroplet
before-patch-firefox.png41.98 KBdroplet

Comments

Status: Needs review » Needs work
Issue tags: -Needs manual testing, -Needs backport to D7

The last submitted patch, fieldset-css.patch, failed testing.

droplet’s picture

droplet commented
Status: Needs work » Needs review
Issue tags: +Needs manual testing, +Needs backport to D7

fieldset-css.patch queued for re-testing.

droplet’s picture

droplet commented
Title: Clean up and add fix multiple line texts problem to fieldset » Clean up and fix fieldset header's multiple line problem
vegantriathlete’s picture

vegantriathlete
he/him/his
Primary language English
Location Lakewood, CO
commented
Status: Needs review » Postponed

@droplet
I am not able to reproduce this. I tested it in both Seven and Bartik. It also appears that the style sheets may have been refactored. Will you take a look at this again and see if you are able to reproduce the problem?

droplet’s picture

droplet commented
Issue tags: -Needs manual testing
StatusFileSize
new align-problem.jpg93.06 KB

@vegantriathlete,

Seems most of the problem are fixed. Just need to align arrow / text well now.

(and backport it )

droplet’s picture

droplet commented
Issue tags: +Needs manual testing

@vegantriathlete,

Seems most of the problem are fixed. Just need to align arrow / text well now.

(and backport it )

vegantriathlete’s picture

vegantriathlete
he/him/his
Primary language English
Location Lakewood, CO
commented
Status: Postponed » Needs work

I would say that a new patch needs to be rolled against the current version of core.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Version: 8.0.x-dev » 7.x-dev
Component: theme system » CSS
Issue summary: View changes
Issue tags: +CSS
StatusFileSize
new Account_settings___Site-Install.png258.63 KB

Apparently this has been fixed already in D8.

valthebald’s picture

valthebald
Location Sofia
commented
Issue tags: -Needs backport to D7 +Needs reroll

Removing 'needs backport', since it's already 7.x issue

r13ose’s picture

r13ose commented

Can the author droplet write a description of how to reproduce the problem?

madhavvyas’s picture

madhavvyas commented
Issue tags: -Needs reroll
StatusFileSize
new fieldset-css-1785978-11.patch2.07 KB

Patch Re-rolled

madhavvyas’s picture

madhavvyas commented
Status: Needs work » Needs review

Status: Needs review » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.