search

mozilla / memchaser

Firefox extension to chase the memory usage and garbage collector activity
https://wiki.mozilla.org/QA/Automation_Services/Projects/Addons/MemChaser
29 stars 23 forks source link

Allow to access the MemChaser data via a button in the main toolbar instead of its own toolbar #176

Open davehunt opened 11 years ago

davehunt commented 11 years ago

The MemChaser addon appears in the main toolbar instead of the addons toolbar in the Australis redesign:

screen shot 2013-08-19 at 12 56 29 pm

If the window is too small, MemChaser is then hidden beneath a >> button, which when clicked only shows the name of the addon but not the content:

screen shot 2013-08-19 at 12 56 47 pm

whimboo commented 11 years ago

As far as I know all of our widgets should be grouped inside our own toolbar. Not sure why all of them are getting put into the navigation bar. I would assume that's a bug in the new SDK UX code.

whimboo commented 11 years ago

Also see https://bugzilla.mozilla.org/show_bug.cgi?id=695913

davehunt commented 11 years ago

I wonder if we should change MemChaser into an icon (we talked about displaying a graph in the past) and provide details when clicked. That way it would take up a lot less space in the UI.

whimboo commented 11 years ago

I would ask someone in the addon sdk channel if that is a temporary solution. I wouldn't spent that much time in UX changes when major changes are coming up anyway on the SDK side.

whimboo commented 10 years ago

This should be fixed by issue #183.

xabolcs commented 10 years ago

I'm just a contributor, but I also vote for button+detail, what @davehunt suggested. For example: a button withRAM: 513 MB where the RAM text is colored based on the GCC and CC times. And the popup panel would contain the currently implemented widget.html content.

The problem with a toolbar is that it will have lot of free space, which is ugly IMHO. The widget.html is no more than 400px wide.

183 is good for introducing a basic Austrlais support, but this issue could be an advanced solution for that.

xabolcs commented 10 years ago

a button with RAM: 513 MB where the RAM text

I meant Resident for the memory string is instead RAM.

whimboo commented 10 years ago

It's something we could think about, yes. We could even make it a configuration setting for the user. Once we add more features it could still be that a toolbar is more helpful. Also personally I want to see all the details including CC/GCC times, so it's hard to make it work for everyone. I will update the summary of this issue to reflect that, and push out the fix to a future release. As you say support for Australis is the most important thing we have to get fixed at the moment.

xabolcs commented 10 years ago

FYI: Bug 907450 - Anchored Panels