Ticket #4345 (closed enhancement: fixed)

Opened 1 year ago

Last modified 10 months ago

On the Comment Editing screens, need a link to View Post that the comment is on

Reported by: dgold Assigned to: westi
Priority: normal Milestone: 2.5
Component: Administration Version:
Severity: normal Keywords: has-patch
Cc:

Description

In the Admin > Comments screens, when editing comments, I would like a Link to View The Post that the Comment is on.

For example -- I'm looking at a Comment, trying to Edit it, and it's not clear what the person was responding to. Where's my post, that has this comment on it? I don't see anything telling me what the original post was. Can you add "View Post" here?

The example screen I'm looking at is: mysite.com/wordpress/wp-admin/comment.php?action=editcomment&c=2239

Attachments

edit-form-comment.php.diff (1.2 kB) - added by mgrouchy on 05/31/07 22:43:41.
Correct Patch for edit-form-comment.php(new)
wp-admin.css.diff (349 bytes) - added by mgrouchy on 05/31/07 22:44:52.
Correct Patch for wp-admin.css(new)

Change History

05/27/07 16:14:32 changed by rob1n

  • owner changed from anonymous to rob1n.

There's a link (the post title) on wp-admin/edit-comments.php with the link (before you edit a comment), if that helps.

But I agree.

05/27/07 16:14:37 changed by rob1n

  • milestone changed from 2.4 to 2.3.

05/30/07 23:00:56 changed by mgrouchy

  • owner changed from rob1n to mgrouchy.

Hey Guys, I'm a summer of code student with WordPress and what I'm working on are enhancements to the Comment panel for Wordpress. I have fixed the issue in this ticket but I forgot to check my patches against the trunk code. If someone can delete the above patches(I don't think I can?) I am gonna upload the correct ones now in a second.

05/30/07 23:21:35 changed by mgrouchy

  • status changed from new to closed.
  • resolution set to fixed.

Okay, Correct Patches added. I have tested it, seems to work fine. I'm calling this fixed. If more any more additions need to be added, let me know.

-Mike

05/30/07 23:28:41 changed by foolswisdom

  • keywords set to has-patch.
  • status changed from closed to reopened.
  • resolution deleted.

mgrouchy: it is marked Fixed by a commiter if they check in your patch (after reviewing it)

05/30/07 23:36:19 changed by rob1n

  • owner changed from mgrouchy to rob1n.
  • status changed from reopened to new.

05/30/07 23:49:50 changed by mgrouchy

Sorry, did not know :) First time posting to Trac! Knew I was going to make more than 1 mistake on here.

05/30/07 23:56:47 changed by rob1n

A few items of concern -- don't take this as personal criticism or anything, just what I would change:

  1. Bad placement -- above there is not the best place to put it, IMO. I would put it at the other side of the submit button.
  2. We try to use semantic HTML in the admin (and everywhere else, really), and <br /><br /> is very presentational.
  3. Don't run dynamic things through the gettext functions. Realistically speaking, the post title is already in the user's language, so that would be useless. The &raquo; would need some treatment, however, so I would use sprintf('%s &raquo;', post_title), or something similar.

05/31/07 03:01:38 changed by mgrouchy

I don't take it personal at all.

1.) The Reason why I put it up there is because that is similar placement to where it is in edit post/edit page. I only moved it to the top, because when I matched it to exact placement with edit page/post it looked weird when you had really long post names. However, it could easily be pushed to the other side of the submit button if thats what the consensus is.

2) I knew the breaks were a bit hacky, once again, they were added because of the problem mentioned above with the really long post names, because the text buts against the text field. However, I can fix that with a tiny bit of css(and I will).

3) Once again, good to know. Will fix that as well.

Anyway, I need to run out now, If i can I will update the patches to reflect changes tonight.If not I will update them Tomorrow.

05/31/07 22:43:02 changed by mgrouchy

Okay, so I took your advice. The line breaks are removed, the link is now lined up with the submit button on the bottom, and I switched the gettext to the formatted string (sprintf).

New diffs are there to replace the old.

Thanks for the input!

05/31/07 22:43:41 changed by mgrouchy

  • attachment edit-form-comment.php.diff added.

Correct Patch for edit-form-comment.php(new)

05/31/07 22:44:52 changed by mgrouchy

  • attachment wp-admin.css.diff added.

Correct Patch for wp-admin.css(new)

06/07/07 02:21:51 changed by rob1n

  • status changed from new to assigned.

09/13/07 21:22:01 changed by ryan

  • milestone changed from 2.3 to 2.4.

12/19/07 07:28:53 changed by rob1n

  • owner deleted.
  • status changed from assigned to new.

12/19/07 18:12:52 changed by westi

  • owner set to westi.
  • status changed from new to assigned.

12/20/07 20:31:47 changed by westi

  • status changed from assigned to closed.
  • resolution set to fixed.

(In [6432]) Add a link to the post a comment is for when editing a comment. Fixes #4345 props mgrouchy,