Skip to content
This repository has been archived by the owner on Jan 13, 2025. It is now read-only.

fix(tabs): Suppress area occupied by scrollbar on platforms that show it #3149

Merged
merged 9 commits into from
Jul 24, 2018

Conversation

kfranqueiro
Copy link
Contributor

Fixes #3059.

@kfranqueiro kfranqueiro requested a review from patrickrodee July 19, 2018 22:21
@codecov-io
Copy link

codecov-io commented Jul 20, 2018

Codecov Report

Merging #3149 into feat/tabs/tabs will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@                Coverage Diff                 @@
##           feat/tabs/tabs    #3149      +/-   ##
==================================================
+ Coverage           98.42%   98.43%   +<.01%     
==================================================
  Files                 113      114       +1     
  Lines                4821     4842      +21     
  Branches              600      603       +3     
==================================================
+ Hits                 4745     4766      +21     
  Misses                 76       76
Impacted Files Coverage Δ
packages/mdc-tab-scroller/constants.js 100% <ø> (ø) ⬆️
packages/mdc-tab-scroller/util.js 100% <100%> (ø)
packages/mdc-tab-scroller/foundation.js 100% <100%> (ø) ⬆️
packages/mdc-tab-scroller/index.js 98.18% <100%> (+0.1%) ⬆️

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 9c75c4d...4be77d8. Read the comment docs.

Copy link
Contributor

@patrickrodee patrickrodee left a comment

Choose a reason for hiding this comment

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

All looks good! Should probably update the README with the new adapter and foundation methods. After that, LGTM!

Copy link
Contributor

@patrickrodee patrickrodee left a comment

Choose a reason for hiding this comment

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

LGTM!

@kfranqueiro kfranqueiro merged commit a842972 into feat/tabs/tabs Jul 24, 2018
@kfranqueiro kfranqueiro deleted the fix/tabs/scrollbar branch July 24, 2018 15:21
@kfranqueiro
Copy link
Contributor Author

After retrying travis at least 3 times and getting spurious unrelated unit test timeouts every time, I've pushed this through regardless.

kfranqueiro pushed a commit that referenced this pull request Jul 27, 2018
WIP fixed bg coloring of icons

fix(tab-indicator): Use absolute positioning (#2547)

WIP start of tab scroller

WIP fixed transition duration

WIP progress on scroller

WIP added demos back

chore(tabs): Removed tab scroller

feat(tabs): Add tab indicator inside tab (#2565)

feat(tab-scroller): Add tab scroller (#2577)

Merge master into feat/tabs/tabs (#3096)

feat(tab): Update tab color and typography (#3108)

docs(tabs): Update metadata and synopses (#3117)

feat(tab): Add MDCTabDimensions computation (#3122)

feat(tab): Emit selection and activation events (#3139)

docs(tabs): Update new READMEs to match standard (#3142)

feat(tab): Give focus to tab when activated (#3164)

feat(tab): Add mixin for parent positioning; Use mixin in tab scroller (#3179)

fix(tabs): Suppress area occupied by scrollbar on platforms that show it (#3149)

fix(tab): Remove extraneous padding from the stacked text label in LTR (#3193)

feat(tabs): Add missing docs and create helper util API (#3194)

Merge master into feat/tabs/tabs (#3227)

feat(tab): Update layout; Add fixed-width mixin; Add min-width class (#3220)

fix(tab-scroller): Fix incorrect animation stopping scroll value in RTL (#3237)

feat(tab-scroller): Add scroll content width method for use in tab bar (#3222)

feat(tab): Remove activation event emitting (#3242)

feat(tab-bar): Add tab bar (#3229)
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.

3 participants