Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hide date and views separator on the watch page when views are hidden #4697

Merged
merged 1 commit into from
Feb 21, 2024

Conversation

absidue
Copy link
Member

@absidue absidue commented Feb 20, 2024

Hide date and views separator on the watch page when views are hidden

Pull Request Type

  • Bugfix

Description

Currently when the Hide View Counts Distraction Free Setting is enabled, the separator between the date and the view count is still shown, this pull request corrects that to hide both at the same time.

Screenshots

before:
before

after:
after

Testing

Turn on the Hide Video Views setting in the General subsection of the Distraction Free Settings and make sure the separator is hidden when the view count is hidden.

Desktop

  • OS: Windows
  • OS Version: 10
  • FreeTube version: b965dc3

@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Feb 20, 2024
@FreeTubeBot FreeTubeBot enabled auto-merge (squash) February 20, 2024 22:25
@FreeTubeBot FreeTubeBot merged commit a6cb20b into FreeTubeApp:development Feb 21, 2024
5 checks passed
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Feb 21, 2024
@absidue absidue deleted the fix-views-separator branch February 21, 2024 17:23
PikachuEXE added a commit to PikachuEXE/FreeTube that referenced this pull request Feb 22, 2024
…n-one-user-playlist-2

* fix/temp/api-ver: (37 commits)
  workaround
  Hide date and views separator on the watch page when views are hidden (FreeTubeApp#4697)
  Translated using Weblate (Ukrainian)
  Translated using Weblate (Polish)
  Translated using Weblate (Estonian)
  Bump webpack from 5.90.1 to 5.90.3 (FreeTubeApp#4696)
  Bump sass from 1.70.0 to 1.71.0 (FreeTubeApp#4693)
  Bump electron from 28.2.2 to 28.2.3 (FreeTubeApp#4692)
  Bump the stylelint group with 1 update (FreeTubeApp#4691)
  Remove unused isLoading from ft-community-post (FreeTubeApp#4684)
  Include swiper version in CSS file name, for cache busting (FreeTubeApp#4685)
  Translated using Weblate (Chinese (Traditional))
  Fix local API erroring when the view count is missing on the channel shorts tab (FreeTubeApp#4689)
  Translated using Weblate (Croatian)
  Translated using Weblate (Polish)
  Fix Hungarian Locale Name (FreeTubeApp#4686)
  Translated using Weblate (Dutch)
  Translated using Weblate (French)
  Translated using Weblate (Spanish)
  Bump marked from 11.2.0 to 12.0.0 (FreeTubeApp#4638)
  ...
PikachuEXE added a commit to PikachuEXE/FreeTube that referenced this pull request Feb 22, 2024
…-user-playlist-2

* development: (37 commits)
  Fix local API search erroring, because the default parameter values were not getting applied (FreeTubeApp#4704)
  Hide date and views separator on the watch page when views are hidden (FreeTubeApp#4697)
  Translated using Weblate (Ukrainian)
  Translated using Weblate (Polish)
  Translated using Weblate (Estonian)
  Bump webpack from 5.90.1 to 5.90.3 (FreeTubeApp#4696)
  Bump sass from 1.70.0 to 1.71.0 (FreeTubeApp#4693)
  Bump electron from 28.2.2 to 28.2.3 (FreeTubeApp#4692)
  Bump the stylelint group with 1 update (FreeTubeApp#4691)
  Remove unused isLoading from ft-community-post (FreeTubeApp#4684)
  Include swiper version in CSS file name, for cache busting (FreeTubeApp#4685)
  Translated using Weblate (Chinese (Traditional))
  Fix local API erroring when the view count is missing on the channel shorts tab (FreeTubeApp#4689)
  Translated using Weblate (Croatian)
  Translated using Weblate (Polish)
  Fix Hungarian Locale Name (FreeTubeApp#4686)
  Translated using Weblate (Dutch)
  Translated using Weblate (French)
  Translated using Weblate (Spanish)
  Bump marked from 11.2.0 to 12.0.0 (FreeTubeApp#4638)
  ...
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.

5 participants