#53 ✓resolved
Laurence Passmore

Main/contextual comment menus out-of-sync with each other

Reported by Laurence Passmore | April 24th, 2008 @ 04:13 PM | in 2.0 Beta Release

Patch attached which syncs the menu options related to comments in both the main and contextual menus. Also disables contextual menu entries when they are not relevant.

Points to note:

1) main menu entries are not disabled when they are not relevant (I couldn't see how to do it)

2) menu items related to deleting comments are called "Remove" because "Delete" is reserved for deleting auctions and you get weird and wonderful disable/enabling effects due to referring to menu items by displayable string(!)

3) is the "View" comment option redundant? It doesn't offer anything beyond "Edit" other than the ability not to edit. :)

Comments and changes to this ticket

  • Morgan Schweers

    Morgan Schweers April 27th, 2008 @ 04:07 AM

    • State changed from “new” to “resolved”


    View isn't completely redundant... You can format your comment as HTML, and it'll view differently than it'll edit.

    However, View should probably be rolled into 'Show Information', which shows the comment as well as a bunch of the other information about the auction.

    This patch is good, and applied as svn [461].

    -- Morgan Schweers, CyberFOX!

  • kaizen

    kaizen December 27th, 2018 @ 11:07 PM

    • Tag set to committed, display, patch

    The article you have shared here very good. This is really interesting information for me. Thanks for sharing!
    subway surfers

  • kaizen

    kaizen December 27th, 2018 @ 11:17 PM

    • no changes were found...

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

Shared Ticket Bins

People watching this ticket