1244268 - JSON Viewer: remove headers tab expand / collapse buttons

Status: RESOLVED FIXED

(DevTools :: JSON Viewer, defect)

Description

[Bryan Clark (DevTools PM) [:clarkbw]]

via bug 1223143 comment 7

> - In the Headers tab, the headings have a [-] icon but can’t be collapsed
> (not that collapsing would be that useful here).

We should just hide the collapse buttons, there is plenty of room to see the headers and the page scrolls otherwise.

Comment 1

[David Walsh :davidwalsh]

Attachment: MozReview Request: Bug 1244268 - Remove expand/collapse headings in JSONView Headers tab

Review commit: https://reviewboard.mozilla.org/r/46863/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/46863/

Comment 2

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Comment on attachment 8741944 [details]
MozReview Request: Bug 1244268 - Remove expand/collapse headings in JSONView Headers tab

https://reviewboard.mozilla.org/r/46863/#review43675

Nice!

Please resolve my comment and I'll r+ this.

We also need a try push for the patch.

Honza

::: devtools/client/jsonview/css/headers-panel.css
(Diff revision 1)
>    color: #565656;
> -}
> -
> -.headersPanelBox .netHeader.twisty {
> -  background-image: url(twisty-open.svg);
> -  background-repeat: no-repeat;

Please remove also twisty-open.svg and twisty.close.svg files - it isn't used anymore.

Honza

Comment 3

[David Walsh :davidwalsh]

Comment on attachment 8741944 [details]
MozReview Request: Bug 1244268 - Remove expand/collapse headings in JSONView Headers tab

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/46863/diff/1-2/

Comment 4

[David Walsh :davidwalsh]

Comment on attachment 8741944 [details]
MozReview Request: Bug 1244268 - Remove expand/collapse headings in JSONView Headers tab

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/46863/diff/2-3/

Comment 5

[David Walsh :davidwalsh]

Comment on attachment 8741944 [details]
MozReview Request: Bug 1244268 - Remove expand/collapse headings in JSONView Headers tab

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/46863/diff/3-4/

Comment 6

[David Walsh :davidwalsh]

Updated!  https://reviewboard.mozilla.org/r/46861/

Comment 7

[David Walsh :davidwalsh]

Comment on attachment 8741944 [details]
MozReview Request: Bug 1244268 - Remove expand/collapse headings in JSONView Headers tab

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/46863/diff/4-5/

Comment 8

[David Walsh :davidwalsh]

Updated again, sorry! Should be set now!

Comment 9

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Comment on attachment 8741944 [details]
MozReview Request: Bug 1244268 - Remove expand/collapse headings in JSONView Headers tab

https://reviewboard.mozilla.org/r/46863/#review43745

Looks good!

Honza

Comment 10

[David Walsh :davidwalsh]

https://reviewboard.mozilla.org/r/46863/#review43745

In looking at the `devtools/client/jsonview/css/moz.build` file, I still see mention of `twisty-closed.svg` and `twist-open.svg`; I presume I missed that and the `moz.build` file must also be updated?

Comment 11

[Jan Honza Odvarko [:Honza] (always need-info? me)]

(In reply to David Walsh :davidwalsh from comment #10)
> https://reviewboard.mozilla.org/r/46863/#review43745
> 
> In looking at the `devtools/client/jsonview/css/moz.build` file, I still see
> mention of `twisty-closed.svg` and `twist-open.svg`; I presume I missed that
> and the `moz.build` file must also be updated?
Ah, yes, both files need to be removed.

Honza

Comment 12

[David Walsh :davidwalsh]

Comment on attachment 8741944 [details]
MozReview Request: Bug 1244268 - Remove expand/collapse headings in JSONView Headers tab

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/46863/diff/5-6/

Comment 13

[Jan Honza Odvarko [:Honza] (always need-info? me)]

https://reviewboard.mozilla.org/r/46863/#review44167

Looks good to me!

Honza

Comment 14

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/ee2fff382352

Comment 15

[Carsten Book [:Tomcat]]

sorry had to back this out in  https://treeherder.mozilla.org/#/jobs?repo=fx-team&revision=f0210b36ff71 since of this patches caused a bustage like :

https://hg.mozilla.org/integration/fx-team/rev/ee2fff382352d57eeb2c7b5ae35c8eaa254e486e

seems the error is ==============================
 05:54:52     INFO -  The error occurred while processing the following file or one of the files it includes:
 05:54:52     INFO -      /builds/slave/fx-team-m64-000000000000000000/build/src/devtools/client/jsonview/css/moz.build
 05:54:52     INFO -  The error occurred when validating the result of the execution. The reported error is:
 05:54:52     INFO -      File listed in FINAL_TARGET_FILES does not exist: /builds/slave/fx-team-m64-000000000000000000/build/src/devtools/client/jsonview/css/twisty-closed.svg
 05:54:52     INFO -  *** Fix above errors and then restart with               "/usr/bin/make -f client.mk build"
05:54:52 INFO - make[3]: *** [conf

Comment 16

[David Walsh :davidwalsh]

Excuse my DevTools+Mercurial workflow noobage but my commit removes said files here:

https://reviewboard.mozilla.org/r/46863/diff/6#2

:Honza : What would cause this build failure?  How would I have not seen this before (is it because I did `./mach build faster` ?

Comment 17

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Solved on IRC, looks like a bug in Reviewboard resulting in a wrong commit. Btw. there are similar bug reports for Reviewboard already: https://bugzilla.mozilla.org/showdependencytree.cgi?id=1255654&hide_resolved=1

Honza

Comment 18

[David Walsh :davidwalsh]

Attachment: MozReview Request: Bug 1244268 - Remove expand/collapse headings in JSONView Headers tab

Review commit: https://reviewboard.mozilla.org/r/49031/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/49031/

Comment 19

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Comment on attachment 8745412 [details]
MozReview Request: Bug 1244268 - Remove expand/collapse headings in JSONView Headers tab

https://reviewboard.mozilla.org/r/49031/#review46081

Comment 20

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/037f55023728

Comment 21

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/037f55023728