Closed Bug 1247696 Opened 8 years ago Closed 8 years ago

UX review of memory panel

Categories

(DevTools :: Memory, defect, P1)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: fitzgen, Assigned: hholmes)

References

(Blocks 1 open bug, )

Details

Joe, as devtools module owner, has made the executive decision that we should do a UX review before turning the memory panel on by default.

Let's use this bug for documenting and addressing issues that come up in the UX review. Depending on how many issues we identify, this may morph into a meta bug.

Helen, what can I do to help the UX review go smoothly? Any idea when we might be able to squeeze this into your TODO list?
Flags: needinfo?(hholmes)
Has STR: --- → irrelevant
Assignee: nobody → hholmes
Flags: needinfo?(hholmes)
I think it might be most useful to set up a meeting and go through the Memory tool together to flesh out this bug. I'll set one up for next week.
Depends on: 1248771
Depends on: 1248988
Depends on: 1249779
Depends on: 1249789
Depends on: 1249791
Depends on: 1249792
Depends on: 1249796
Depends on: 1249798
We should be sure to update the docs for all these changes
Helen, are there any other actions items you'd like to see come out of this? Should we resolve this bug?
Flags: needinfo?(hholmes)
(In reply to Nick Fitzgerald [:fitzgen] [⏰PDT; UTC-7] from comment #3)
> Helen, are there any other actions items you'd like to see come out of this?
> Should we resolve this bug?
In Slack Fang and I have been working on some new loading states, this being the "long" version:
- https://drive.google.com/a/mozilla.com/file/d/0B1V1eo-I8g3dMWNYeVJJaXRWS0k/view

I'd originally asked her to come up with the longer loading state to capture some of the longer load times for the memory panel, but I know you've put work in to making the memory tool much faster. Is this work still worth doing? Would this bug be the bug to capture implementing that work if it is?
Flags: needinfo?(hholmes) → needinfo?(nfitzgerald)
"snapchat" lol

I like that animation a lot, very cool!

I think we can file a new bug for that, and close this one. Filed bug 1279116 for that. I think it is still worthwhile -- we've made lots of performance improvements but we're dealing with computations that are pretty heavy by nature, so I don't think we will ever reach something close to instant.
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: needinfo?(nfitzgerald)
Resolution: --- → FIXED
(In reply to Helen V. Holmes (:helenvholmes) (:✨)(pls ni?) from comment #4)
> (In reply to Nick Fitzgerald [:fitzgen] [⏰PDT; UTC-7] from comment #3)
> > Helen, are there any other actions items you'd like to see come out of this?
> > Should we resolve this bug?
> In Slack Fang and I have been working on some new loading states, this being
> the "long" version:
> -
> https://drive.google.com/a/mozilla.com/file/d/0B1V1eo-I8g3dMWNYeVJJaXRWS0k/
> view
> 

This animation is amazing.
> I'd originally asked her to come up with the longer loading state to capture
> some of the longer load times for the memory panel, but I know you've put
> work in to making the memory tool much faster. Is this work still worth
> doing? Would this bug be the bug to capture implementing that work if it is?
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.