Skip to content
This repository was archived by the owner on Jan 11, 2023. It is now read-only.

Add footer tabs to sources pane for outline view#3087

Merged
jasonLaster merged 4 commits into
firefox-devtools:masterfrom
amelzer:outline/add-footer-tabs
Jun 5, 2017
Merged

Add footer tabs to sources pane for outline view#3087
jasonLaster merged 4 commits into
firefox-devtools:masterfrom
amelzer:outline/add-footer-tabs

Conversation

@amelzer
Copy link
Copy Markdown
Contributor

@amelzer amelzer commented Jun 5, 2017

Associated Issue: #2901

As part of the outline view, the SVG button toggling the outline view of the sources pane has been replaced with tabs

Summary of Changes

  • Replaced the toggle SVG with footer tabs for sources and outline view

Test Plan

  • Activating "Outline" in Settings shows the footer
  • Deactivating "Outline" in settings hides the footer
  • Clicking the "Outline View" tab navigates to the outline view
  • Clicking the "Sources View" tab navigates to the outline view

Screenshots/Videos (OPTIONAL)

debugger

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 5, 2017

Codecov Report

Merging #3087 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3087   +/-   ##
=======================================
  Coverage   66.15%   66.15%           
=======================================
  Files          74       74           
  Lines        2603     2603           
  Branches      527      527           
=======================================
  Hits         1722     1722           
  Misses        881      881

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9d582f4...f0c3d16. Read the comment docs.

Copy link
Copy Markdown
Contributor

@codehag codehag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi! Awesome work, Looks really great! Thank you so much 😄

I noticed a bit of flashing when switching from outline to source tree, do you see it too? I think it was introduced earlier, just want to confirm. This only happens when the source tree has a text that is too long for the panel. It looks like the height is jumping up when the source panel is being hidden. I don't think its related to this pr though, just wanted to check if you had any ideas.

@amelzer
Copy link
Copy Markdown
Contributor Author

amelzer commented Jun 5, 2017

I accidentally broke the tab page with my css. Will add a fix.

screen shot 2017-06-05 at 12 54 34

@amelzer amelzer changed the title Add footer tabs to sources pane for outline view [WiP] Add footer tabs to sources pane for outline view Jun 5, 2017
@amelzer amelzer changed the title [WiP] Add footer tabs to sources pane for outline view Add footer tabs to sources pane for outline view Jun 5, 2017
@AnshulMalik
Copy link
Copy Markdown
Contributor

Yes @codehag, The flicker was introduced when this feature was first added. I didn't find any good solution to resolve the issue.

Copy link
Copy Markdown
Contributor

@jasonLaster jasonLaster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

This looks great. I love the attention to detail

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants