Opened 10 years ago
Closed 10 years ago
#4345 closed enhancement (fixed)
On the Comment Editing screens, need a link to View Post that the comment is on
Reported by: | dgold | Owned by: | westi |
---|---|---|---|
Milestone: | 2.5 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Administration | Keywords: | has-patch |
Focuses: | 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 (2)
Change History (17)
#3
@mgrouchy
10 years ago
- 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.
#4
@mgrouchy
10 years ago
- Resolution set to fixed
- Status changed from new to closed
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
#5
@foolswisdom
10 years ago
- Keywords has-patch added
- Resolution fixed deleted
- Status changed from closed to reopened
mgrouchy: it is marked Fixed by a commiter if they check in your patch (after reviewing it)
#7
@mgrouchy
10 years ago
Sorry, did not know :) First time posting to Trac! Knew I was going to make more than 1 mistake on here.
#8
@rob1n
10 years ago
A few items of concern -- don't take this as personal criticism or anything, just what I would change:
- 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.
- We try to use semantic HTML in the admin (and everywhere else, really), and <br /><br /> is very presentational.
- 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 » would need some treatment, however, so I would use sprintf('%s »', post_title), or something similar.
#9
@mgrouchy
10 years ago
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.
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.