My favorites | Sign in
Project Home Wiki Issues
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 5454: Scroll to see the props when a group is expanded inside the Computed side panel
1 person starred this issue and may be notified of changes. Back to list
 
Reported by sebastia...@gmx.de, May 10, 2012
What should be implemented/improved?
If a group is expanded, the content should scroll up to show content of the group.
But the content should scroll just the necessary amount of space, e.g. if the group has just one entry it scrolls up to show that one entry, not more.

Test case:
1. Open Firebug on this page
2. Switch to the HTML panel and there to the Computed side panel
3. Resize Firebug to about 400px height
4. Click the titles of all groups except the "Text" group
   => The groups should be collapsed
5. Scroll down until you see the "Background" group title
6. Click the title of the "Background" group
   => The group should be expanded and scrolled so that you see the last property of the group at the bottom of the panel (not implemented yet)
7. Scroll a bit down so that you see the next group's title ("Box Model")
8. Click the title of that group
   => The group should be expanded and scrolled down so that the title is displayed at the top of the Computed side panel. (not implemented yet)

Sebastian
May 10, 2012
Project Member #1 odva...@gmail.com
Yep, this is exactly what I had in mind. Note that this is exactly how tree views usually work (e.g. my project Navigator in Eclipse, or File explorer in Windows)

Honza
May 10, 2012
#2 sebastia...@gmx.de
> Note that this is exactly how tree views usually work (e.g. my project Navigator in 
> Eclipse, or File explorer in Windows)
Honestly, I never realized that.

Sebastian

May 10, 2012
#3 sebastia...@gmx.de
(No comment was entered for this change.)
Blocking: 5450
May 11, 2012
#4 sebastia...@gmx.de
Done in https://github.com/firebug/firebug/commit/bc4110774ca4a71bb580a67d8d5ff07d2b704fea.

Sebastian
Status: Commit
Labels: FBTest-wanted
May 11, 2012
Project Member #5 odva...@gmail.com
Works great!

I think it could also work for expanded CSS rules.

Honza
May 11, 2012
Project Member #6 odva...@gmail.com
This issue (one of 25) has been fixed in Firebug 1.10a8
http://getfirebug.com/releases/firebug/1.10/firebug-1.10.0a8.xpi

Please try it and let us know whether it works for you.

Thanks!
Honza

Status: Fixed
Labels: fixed-1.10-a8
May 12, 2012
#7 sebastia...@gmx.de
> I think it could also work for expanded CSS rules.
Put this into  issue 5459 .

Sebastian
Status: Verified
Jul 12, 2012
Project Member #8 sebastia...@gmail.com
(No comment was entered for this change.)
Blocking: fbug:5450
Jul 12, 2012
Project Member #9 sebastia...@gmail.com
Added note about this to https://getfirebug.com/wiki/index.php/Computed_Side_Panel.

Sebastian
Owner: sebastia...@gmail.com
Labels: doc-available
Blockedon: -fbug:2916 fbug:2916
Jul 14, 2013
Project Member #10 sebastia...@gmail.com
Added FBTest for this in https://github.com/firebug/firebug/commit/7284a4a244ca33ff5f5a749b66eff226d84c7a9d.

Sebastian
Labels: -FBTest-wanted FBTest-available
Sign in to add a comment

Powered by Google Project Hosting