Make WordPress Core

Opened 17 years ago

Closed 17 years ago

#5985 closed defect (bug) (fixed)

Styling Manage Comments page

Reported by: iammattthomas's profile iammattthomas Owned by:
Milestone: 2.5 Priority: normal
Severity: normal Version: 2.5
Component: General Keywords: has-patch
Focuses: Cc:

Description

CSS for Manage Comments page.

Attachments (3)

5985.diff (6.3 KB) - added by iammattthomas 17 years ago.
5985-2.diff (2.3 KB) - added by iammattthomas 17 years ago.
Move updated div above .wrap for consistency with other pages
5985-2.2.diff (2.7 KB) - added by iammattthomas 17 years ago.

Download all attachments as: .zip

Change History (8)

@iammattthomas
17 years ago

#1 follow-up: @iammattthomas
17 years ago

As far as I can tell, there's no way to split onelongwordwithnospaces with CSS (currently). So the URL will need to be trimmed down in the source.

Added some code to apply a "class='current'" to the Detail View/List View switcher, might want to look at that and make sure it's kosher (it works, but I don't know enough PHP to know if my style is good).

Where it says "From [post name], [Date] at [Time]," that should read "From [post name], [N hours/minutes ago]" for comments posted today; and "From [post name] at [time]" for all others. The date should be taken out since it's got its own column.

The separate moderation.php page, as far as I can tell, should be deprecated in favor of edit-comments.php?comment_status=moderated. Happy Cog didn't provide a design for the moderation page so as I understand it, all moderation should be handled via this new form.

#2 @ryan
17 years ago

(In [7005]) Manage comments styling from mt. see #5985

#3 in reply to: ↑ 1 @ryan
17 years ago

Replying to iammattthomas:

As far as I can tell, there's no way to split onelongwordwithnospaces with CSS (currently). So the URL will need to be trimmed down in the source.

I'll trim it.

Added some code to apply a "class='current'" to the Detail View/List View switcher, might want to look at that and make sure it's kosher (it works, but I don't know enough PHP to know if my style is good).

Looks good.

Where it says "From [post name], [Date] at [Time]," that should read "From [post name], [N hours/minutes ago]" for comments posted today; and "From [post name] at [time]" for all others. The date should be taken out since it's got its own column.

I'll add the human time. The separate date column is for the comment's date. The date in the from line is the post's date. I think we need both.

The separate moderation.php page, as far as I can tell, should be deprecated in favor of edit-comments.php?comment_status=moderated. Happy Cog didn't provide a design for the moderation page so as I understand it, all moderation should be handled via this new form.

I think it's going away. I'll remove it once I finish hooking up all the buttons in edit-comments.php and we fix the AJAX interactions. Akismet will also need to change to put the recheck queue link on edit-comments.php.

@iammattthomas
17 years ago

Move updated div above .wrap for consistency with other pages

#4 @iammattthomas
17 years ago

Added new patch that moves .updated above .wrap so it doesn't interfere with the .view-switch.

#5 @ryan
17 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [7011]) Manage comments styling from mt. fixes #5985

Note: See TracTickets for help on using tickets.