Jan Honza Odvarko [:Honza] (always need-info? me) Reporter
	Description • 8 years ago

Side panel tabs are having wrong height in Firebug theme. See the attached screenshot.

Honza

	Jan Honza Odvarko [:Honza] (always need-info? me) Reporter
	Comment 1 • 8 years ago

A few comments.

General styles for the tab control are here:
https://dxr.mozilla.org/mozilla-central/source/devtools/client/shared/components/tabs/tabs.css

Styles for the tabbar (build on top of tabs) are here:
https://dxr.mozilla.org/mozilla-central/source/devtools/client/shared/components/tabs/tabbar.css

Styles specific to the Inspector panel are here:
https://dxr.mozilla.org/mozilla-central/source/devtools/client/themes/inspector.css

We need to figure out what is the best place for the fix (depending how generic is it).

Honza

	Ruturaj Vartak Assignee
	Comment 2 • 8 years ago

In linux atleast this small change works like a charm.

	Ruturaj Vartak Assignee
	Comment 3 • 8 years ago

Firebug theme:
- fixed inspector side panel tab height
- fixed network monitor side panel tab height

	Ruturaj Vartak Assignee
	Comment 4 • 8 years ago

Tab heights in inspector and network monitor sidebar - before and after fix (Linux)

	Jan Honza Odvarko [:Honza] (always need-info? me) Reporter
	Comment 5 • 8 years ago

Comment on attachment 8815610 [details] [diff] [review]
fix-1320304-2.patch

Review of attachment 8815610 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me, thanks for working on this!

Hozna

	Ruturaj Vartak Assignee
	Comment 6 • 8 years ago

Hey Honza,
Is "checkin-needed" keyword applicable now ?

	Jan Honza Odvarko [:Honza] (always need-info? me) Reporter
	Comment 7 • 8 years ago

(In reply to Ruturaj Vartak from comment #6)
> Hey Honza,
> Is "checkin-needed" keyword applicable now ?
Yes

Honza

	Pulsebot
	Comment 8 • 8 years ago

Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/0c212b76bca6
Firebug theme - Wrong height of Inspector, Network side panel tabs. r=Honza

	Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)
	Comment 9 • 8 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/0c212b76bca6

	Mohammad Maruf Rahman
	Comment 10 • 8 years ago

I have reproduced this bug with Nightly 53.0a1 (2016-11-25) (64-bit) on windows 7 , 64 Bit ! 

This bug's fix is verified with latest Nightly

Build ID          20161210030206
User Agent        Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:53.0) Gecko/20100101 Firefox/53.0 

[bugday-20161207]

	Farhadur Reja Fahim
	Comment 11 • 7 years ago

I have reproduced this bug with Nightly 53.0a1 (2016-11-25) (64-bit) on Ubuntu 16.04
The bug's fix is now verified on latest Nightly 55.0a1.

Nightly 55.0a1:

Build ID 	20170420100256
User Agent 	Mozilla/5.0 (X11; Linux x86_64; rv:55.0) Gecko/20100101 Firefox/55.0

[testday: 20170421]