WordPress.org

Make WordPress Core

Opened 2 years ago

Last modified 4 months ago

#25060 assigned defect (bug)

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

Reported by: c3mdigital Owned by: helen
Milestone: Future Release Priority: normal
Severity: normal Version: 3.1
Component: Quick/Bulk Edit Keywords: has-patch
Focuses: ui, javascript, administration 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 2 years ago.
25060.2.patch (1.5 KB) - added by c3mdigital 2 years ago.
Fixes inline background-color and classes
25060.3.patch (1.4 KB) - added by c3mdigital 2 years ago.
tabs not spaces
25060.4.patch (3.0 KB) - added by c3mdigital 2 years ago.
Replace javascript hex color background animations with class change animations
25060.diff (2.7 KB) - added by helen 7 months ago.

Download all attachments as: .zip

Change History (19)

@c3mdigital2 years ago

comment:1 @c3mdigital2 years ago

  • Keywords needs-refresh removed

@c3mdigital2 years ago

Fixes inline background-color and classes

comment:2 @nacin2 years ago

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

@c3mdigital2 years ago

tabs not spaces

comment:3 follow-up: @helen2 years ago

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

comment:4 in reply to: ↑ 3 ; follow-up: @c3mdigital2 years 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 2 years ago by c3mdigital (next)

comment:5 in reply to: ↑ 4 @helen2 years ago

Replying to c3mdigital:

Yes please pretty please.

@c3mdigital2 years ago

Replace javascript hex color background animations with class change animations

comment:6 @helen23 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 @nacin22 months ago

  • Milestone changed from 3.7 to Future Release

Moving based on Helen's observations.

comment:8 @helen22 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 @helen7 months ago

  • Milestone changed from Future Release to 4.2

@helen7 months ago

comment:10 @helen7 months 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 @helen7 months 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.

comment:12 follow-up: @DrewAPicture5 months ago

Based on [31181], looks like this issue still hasn't been resolved for comments quick edit. @helen: Did you still want to try to resolve this for 4.2?

comment:13 in reply to: ↑ 12 @helen5 months ago

Replying to DrewAPicture:

Did you still want to try to resolve this for 4.2?

That's what #31006 is for - this relies on that.

comment:14 @helen4 months ago

  • Focuses ui javascript administration added
  • Milestone changed from 4.2 to Future Release

Still blocked, too much to be doing at this point in 4.2. A comments-list-table-only solution could be done, but I think that would put us in a bad position later.

Note: See TracTickets for help on using tickets.