Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Feb 5, 2021

Fixes #2616

@ghost ghost self-requested a review February 5, 2021 00:03
Copy link
Collaborator

@vdonato vdonato left a comment

Choose a reason for hiding this comment

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

Also think it'd be relatively easy to add a unit test for this one, but should be good to go after that

@ghost
Copy link
Author

ghost commented Feb 5, 2021

@vdonato Ok, I added a snapshot test, which I think is better suited to testing appearance.

@ghost ghost merged commit 56475c9 into streamlit:develop Feb 5, 2021
@ghost ghost deleted the fix-progress-bar-alignment branch February 5, 2021 23:53
tconkling added a commit that referenced this pull request Feb 8, 2021
* develop:
  Autofocus "clear cache" button (#2739)
  Fix checkbox spacing (#2738)
  Fix progress bar alignment (#2734)
  Fix select menu not autoselecting first item (#2732)
  Show user command line in disconnect dialog (#2735)
  Up version to 0.76.0
  Update change log
  Improve usage warning wording and formatting
  Finally remove beta versions of set_page_config and color_picker (#2711)
  Finally remove beta versions of set_page_config and color_picker (#2711)
  Clean up file update calls (#2597)
tconkling added a commit to tconkling/streamlit that referenced this pull request Feb 11, 2021
* develop:
  Slider thumb values are always visible (streamlit#2724)
  Update component-template submodule to latest (streamlit#2767)
  `allow_multiple_files` -> `accept_multiple_files` (streamlit#2761)
  Autofocus "clear cache" button (streamlit#2739)
  Fix checkbox spacing (streamlit#2738)
  Fix progress bar alignment (streamlit#2734)
  Fix select menu not autoselecting first item (streamlit#2732)
  Show user command line in disconnect dialog (streamlit#2735)
  Up version to 0.76.0
  Update change log
  Improve usage warning wording and formatting
  Finally remove beta versions of set_page_config and color_picker (streamlit#2711)
  Finally remove beta versions of set_page_config and color_picker (streamlit#2711)
  Clean up file update calls (streamlit#2597)
schaumb pushed a commit to FloWide/streamlit that referenced this pull request Feb 22, 2021
* fix progress bar alignment

* add snapshot test to progress bar

* add snapshot
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Visual regression: progress bar alignment

1 participant