JAL-2405: Protein specific colours enabled in selected group colour menu for...


CR-JAL-22 3

Keyboard shortcuts  
  • Summarize the review outcomes (optional)


    Warning: no files are visible, they have all been filtered.
    Participant Role Time Spent Comments Latest Comment
    Author 25m 2 Added a further check, with a unit test, also in Alignmen...
    Reviewer - Complete 9m 1 If it is possible to get a circular reference chain as in...
    Total   34m 3  


    1.Import a nucleotide alignment
    2. make a selection
    3. right click and Selection->new group->colour menu

    At the very least, Conservation colouring should not be available since it is not computable for nucleotide sequences.

    Branches in review


    Issues Raised From Comments

    Key Summary State Assignee

    General Comments

    There are no general comments on this review.
    /src/jalview/appletgui/APopupMenu.java Changed
    Open in IDE #permalink
    /src/jalview/appletgui/AlignFrame.java Changed
    /src/jalview/datamodel/SequenceGroup.java Changed 3
    /src/jalview/gui/PopupMenu.java Changed
    Open in IDE #permalink
    /src/.../viewmodel/AlignmentViewport.java Changed
    /test/.../datamodel/SequenceGroupTest.java Changed
    /test/jalview/gui/AlignViewportTest.java Changed

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time against