On roll over, Super-User quote should be static

VERIFIED FIXED in 1.0

Status

P1
major
VERIFIED FIXED
8 years ago
8 years ago

People

(Reporter: lmesa, Assigned: adam)

Tracking

unspecified
x86
Mac OS X

Details

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
Created attachment 523476 [details]
static box

The quote box for each super user should be static while the user remains highlighted. Upon navigation away from the mark, the box should go away. 

See image attached.
(Assignee)

Comment 1

8 years ago
Fixed here: https://github.com/mozilla/markup/commit/91921f4bd0f528029c4ca12c923a498035bb4f9d#diff-32
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
(Reporter)

Comment 2

8 years ago
This is not working on our stage server. 

Test for yourself here: https://markup.allizom.org/accounts/login/ u: admin p: cH568Efruxas
Severity: normal → major
Status: RESOLVED → REOPENED
Priority: -- → P1
Resolution: FIXED → ---
Target Milestone: --- → 1.0
(In reply to comment #2)
> This is not working on our stage server. 
> 
> Test for yourself here: https://markup.allizom.org/accounts/login/ u: admin p:
> cH568Efruxas

:-/ this is a public bug. I changed the admin password and updated it in the internal markup docs. Please share the credentials by email instead if needed.
(Reporter)

Comment 4

8 years ago
(In reply to comment #3)
> (In reply to comment #2)
> > This is not working on our stage server. 
> > 
> > Test for yourself here: https://markup.allizom.org/accounts/login/ u: admin p:
> > cH568Efruxas
> 
> :-/ this is a public bug. I changed the admin password and updated it in the
> internal markup docs. Please share the credentials by email instead if needed.

I'm a knuckle head! Sorry about that. Adam--sending you new credentials by email now.
(Assignee)

Comment 5

8 years ago
Laura, what's the bug here? 

Why are you directing me to the admin login? The contributor, or superuser, marks shouldn't ever show up in the admin tool. I can view this one:

https://markup.allizom.org/en/#/linear/dKax

And the quote from Matt behaves as I expected.
(Reporter)

Comment 6

8 years ago
The bug here is that the quote should be present at all times when its in the center of the viewing pane. Right now, its only present on rollover, so the discover-ability is low. We want to make it obvious that these marks are different, so highlighting is one way, but the quote is also key.
(Assignee)

Comment 7

8 years ago
Sorry, I misunderstood this bug initially. I need to talk to Henry about this change before I finish my fix, but it's underway.
(Reporter)

Updated

8 years ago
Assignee: adam → jbresnik

Updated

8 years ago
Assignee: jbresnik → adam
:lmesa, if you've got a moment can you verify the fix?
Status: REOPENED → RESOLVED
Last Resolved: 8 years ago8 years ago
Resolution: --- → FIXED
(Reporter)

Comment 10

8 years ago
Works for me!
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.