KEMBAR78
core: fix the header pagination by stephanwlee · Pull Request #4849 · tensorflow/tensorboard · GitHub
Skip to content

Conversation

@stephanwlee
Copy link
Contributor

TensorBoard customizes the material tab header component to have less
visual jank and have consistent behavior as the Polymer based
TensorBoard. This caused the scroll calculation to be incorrect (about
36 * 2 pixels) and a plugin like "text" where the tab label is very
short caused the bug to manifest quite obviously.

This change fixes that by changing how the mat-header is styled by
overriding few of its internals.

Fixes #4841.

TensorBoard customizes the material tab header component to have less
visual jank and have consistent behavior as the Polymer based
TensorBoard. This caused the scroll calculation to be incorrect (about
36 * 2 pixels) and a plugin like "text" where the tab label is very
short caused the bug to manifest quite obviously.

This change fixes that by changing how the mat-header is styled by
overriding few of its internals.

Fixes tensorflow#4841.
@stephanwlee stephanwlee requested a review from ericdnielsen April 9, 2021 03:03
@google-cla google-cla bot added the cla: yes label Apr 9, 2021
@stephanwlee
Copy link
Contributor Author

Gentle ping! :D

@stephanwlee stephanwlee requested review from psybuzz and removed request for ericdnielsen April 15, 2021 21:58
Copy link
Contributor

@psybuzz psybuzz left a comment

Choose a reason for hiding this comment

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

lgtm! I'm assuming that all TB variants have a header color of $tb-primary, 700

@stephanwlee
Copy link
Contributor Author

all TB variants have a header color of $tb-primary, 700

Currently, they are all set to "primary" at the Material level.

@stephanwlee stephanwlee merged commit 4b40bb5 into tensorflow:master Apr 22, 2021
@stephanwlee stephanwlee deleted the fix_header branch April 22, 2021 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Plugin in header bar can be hidden with no chevron to scroll

2 participants