WordPress.org

Make WordPress Core

Opened 19 months ago

Last modified 6 weeks ago

#25060 assigned defect (bug)

Quick Edit Comments inline edit save breaks the alternating table row backgrounds

Reported by: c3mdigital Owned by: helen
Milestone: 4.2 Priority: normal
Severity: normal Version: 3.1
Component: Quick/Bulk Edit Keywords: has-patch
Focuses: Cc:

Description

Related: #24951
Related: #25059

When quick editing comments and saving the single_row method always uses the same classes "even thread-even". Additionally a css background color gets added to the table row inline.

This patch fixes the class issues. Still need a patch for the background color.

Attachments (5)

25060.patch (1.0 KB) - added by c3mdigital 19 months ago.
25060.2.patch (1.5 KB) - added by c3mdigital 19 months ago.
Fixes inline background-color and classes
25060.3.patch (1.4 KB) - added by c3mdigital 19 months ago.
tabs not spaces
25060.4.patch (3.0 KB) - added by c3mdigital 19 months ago.
Replace javascript hex color background animations with class change animations
25060.diff (2.7 KB) - added by helen 6 weeks ago.

Download all attachments as: .zip

Change History (16)

@c3mdigital19 months ago

comment:1 @c3mdigital19 months ago

  • Keywords needs-refresh removed

@c3mdigital19 months ago

Fixes inline background-color and classes

comment:2 @nacin19 months ago

  • Milestone changed from Awaiting Review to 3.7
  • Owner set to helen
  • Status changed from new to assigned

@c3mdigital19 months ago

tabs not spaces

comment:3 follow-up: @helen19 months ago

We have hex values set in JS? :( :( :(

comment:4 in reply to: ↑ 3 ; follow-up: @c3mdigital19 months ago

Replying to helen:

We have hex values set in JS? :( :( :(

Yes I almost cried too. They are really just used for the green background on un approved comments. Maybe we should be adding a class and update the css?

Version 0, edited 19 months ago by c3mdigital (next)

comment:5 in reply to: ↑ 4 @helen19 months ago

Replying to c3mdigital:

Yes please pretty please.

@c3mdigital19 months ago

Replace javascript hex color background animations with class change animations

comment:6 @helen18 months ago

Been fighting with this - latest patch breaks the visual change of parent comment approval when you approve and reply inline.

More than that, though, the zebra striping has issues other than just editing inline - replying inline causes everything to be off. We can still fix this piece of the problem, but just wanted to note that there's more to the entire picture.

comment:7 @nacin17 months ago

  • Milestone changed from 3.7 to Future Release

Moving based on Helen's observations.

comment:8 @helen17 months ago

This is a place where using :nth-child selectors in CSS would work out way better. IE7 and 8 would just not get zebra striping anymore.

comment:9 @helen6 weeks ago

  • Milestone changed from Future Release to 4.2

@helen6 weeks ago

comment:10 @helen6 weeks ago

Opened #31006 to handle the background hex values in JS thing, as it happens outside of comments as well - will continue off of your start, c3mdigital.

comment:11 @helen6 weeks ago

In 31181:

List tables: Use CSS :nth-child() selectors for zebra striping.

Note that this does not fix issues related to comment quick edit. Internal linking also continues to use the .alternate class for now. IE8 and below gracefully degrade by not having zebra striping.

There is some hoop jumping with adding an extra table row to maintain zebra striping during quick edit. Documenting that here for future reference; it is also in the inline documentation.

fixes #30981 and #26060. see #25060.

Note: See TracTickets for help on using tickets.