Annotation feedback edit/delete buttons not styled correctly

Bug #1620119 reported by Robert Lyon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
High
Robert Lyon

Bug Description

When comparing the edit/delete buttons on annotation feedback they are different to the same set of buttons on comments, when comments are a block on the page rather than at foot of the page.

See attached screenshot

The issues I see are:

1) The first annotation feedback's edit button is floating in space rather than being anchored to a line

2) When we have both edit and delete buttons they are not horizontally next to each other.

Also another issue is an admin can't delete their own feedback from their own page's annotation block.

In artefact/annotation/lib.php there is the following code:

    else if (($candelete || $item->isauthor) && !$is_export_preview && !$isadminfeedback) {
        // If the auther was admin/staff and not the owner of the annotation,
        // the feedback can't be deleted.

But it is not correct when the owner is also an administrator.

Revision history for this message
Robert Lyon (robertl-9) wrote :
Revision history for this message
Robert Lyon (robertl-9) wrote :

I've marked this bug as 'high' because it would be good to fix this asap as part of 16.10rc2

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "master" branch: https://reviews.mahara.org/6922

Robert Lyon (robertl-9)
Changed in mahara:
status: Confirmed → In Progress
assignee: nobody → Robert Lyon (robertl-9)
Robert Lyon (robertl-9)
summary: - Annotation feedback deit/delete buttons not styled correctly
+ Annotation feedback edit/delete buttons not styled correctly
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/6922
Committed: https://git.mahara.org/mahara/mahara/commit/5f70a878fc0fc0ba2a87bd6b71bd4f752ac00e2f
Submitter: Son Nguyen (<email address hidden>)
Branch: master

commit 5f70a878fc0fc0ba2a87bd6b71bd4f752ac00e2f
Author: Robert Lyon <email address hidden>
Date: Fri Sep 2 15:17:47 2016 +1200

Bug 1620119: Fix for the brokenness in annotation feedback list

Things fixed here:

1) The floating in space button for first feedback entry
2) The poor alignment for the annotation edit + delete buttons combo
3) Allowing an admin to delete their own feedback on their own
annotation

behatnotneeded

Change-Id: I829280d89ca0020b2895120124ac59507285bf71
Signed-off-by: Robert Lyon <email address hidden>

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "16.10_STABLE" branch: https://reviews.mahara.org/6955

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/6955
Committed: https://git.mahara.org/mahara/mahara/commit/ff2bee5b291082bbb973b4c46b43d3a7c92b45d4
Submitter: Robert Lyon (<email address hidden>)
Branch: 16.10_STABLE

commit ff2bee5b291082bbb973b4c46b43d3a7c92b45d4
Author: Robert Lyon <email address hidden>
Date: Fri Sep 2 15:17:47 2016 +1200

Bug 1620119: Fix for the brokenness in annotation feedback list

Things fixed here:

1) The floating in space button for first feedback entry
2) The poor alignment for the annotation edit + delete buttons combo
3) Allowing an admin to delete their own feedback on their own
annotation

behatnotneeded

Change-Id: I829280d89ca0020b2895120124ac59507285bf71
Signed-off-by: Robert Lyon <email address hidden>
(cherry picked from commit 5f70a878fc0fc0ba2a87bd6b71bd4f752ac00e2f)

Robert Lyon (robertl-9)
Changed in mahara:
status: In Progress → Fix Committed
milestone: 16.10rc2 → 16.10.0
Robert Lyon (robertl-9)
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.