Make WordPress Core

Opened 13 years ago

Closed 12 years ago

#26648 closed defect (bug) (duplicate)

Div is not cleared after floated element in /wp-admin/themes.php

Reported by: ankit.gade@… Owned by:
Priority: normal Milestone:
Component: Themes Version: 3.8
Severity: normal Keywords: has-patch
Cc: Focuses: ui, administration

Description

Div (div.theme-browser) is not cleared after the floated element h2 ( "Themes" heading ). Because of this it will not take any margin or padding properly, if needs to apply in future.
So div.theme-browser needs to be cleared after floated element by adding class "clear".

Attachments (3)

themes1.jpg (170.4 KB ) - added by ankit.gade@… 13 years ago.
Screenshot of themes heading
themes2.jpg (152.8 KB ) - added by ankit.gade@… 13 years ago.
Screenshot of non cleared div
appearance-themes.patch (277 bytes ) - added by ankit.gade@… 13 years ago.
Patch to clear div by adding class clear

Download all attachments as: .zip

Change History (13)

@ankit.gade@…
13 years ago

Screenshot of themes heading

@ankit.gade@…
13 years ago

Screenshot of non cleared div

@ankit.gade@…
13 years ago

Patch to clear div by adding class clear

#1 @ankit.gade@…
13 years ago

  • Keywords has-patch added

#2 @ankit.gade@…
13 years ago

  • Cc ankit.gade@… added

#3 @SergeyBiryukov
13 years ago

  • Keywords ui-focus added

#4 @ankit.gade@…
13 years ago

  • Version 3.8trunk

#5 follow-up: @SergeyBiryukov
13 years ago

  • Version trunk3.8

Version number indicates the earliest affected version.

Because of this it will not take any margin or padding properly, if needs to apply in future.

As long as everything is currently displayed as expected, I don't see this as an issue. If we decide to add a margin or padding in the future, we can add the clear class at the same time.

#6 @ankit.gade@…
13 years ago

Last edited 13 years ago by ankit.gade@… (previous) (diff)

#7 in reply to: ↑ 5 @ankit.gade@…
13 years ago

Replying to SergeyBiryukov:

Version number indicates the earliest affected version.

Because of this it will not take any margin or padding properly, if needs to apply in future.

As long as everything is currently displayed as expected, I don't see this as an issue. If we decide to add a margin or padding in the future, we can add the clear class at the same time.

That is fine but it is the css concept that whenever there is an floated element, it should clear immediately just before non-floated element; otherwise its parent container would show height and width as 0px. Also floated element does not occupy space, but its neighbor should start next to that element, in current scenario it is also covering over floated h2 element.

As it is WordPress, we all expect it to follow all the standards. :)

Last edited 13 years ago by ankit.gade@… (previous) (diff)

#9 @nacin
13 years ago

  • Component AppearanceThemes
  • Focuses administration added

#10 @obenland
12 years ago

  • Milestone Awaiting Review
  • Resolutionduplicate
  • Status newclosed

Duplicate of #26646.

Note: See TracTickets for help on using tickets.