Skip to content
This repository has been archived by the owner on May 28, 2024. It is now read-only.

Cannot see controls with white background videos #146

Open
KJ2ME opened this issue Dec 21, 2019 · 5 comments · May be fixed by #196
Open

Cannot see controls with white background videos #146

KJ2ME opened this issue Dec 21, 2019 · 5 comments · May be fixed by #196

Comments

@KJ2ME
Copy link

KJ2ME commented Dec 21, 2019

Image of the error: https://i.imgur.com/A4TfNsp.png

Steps to reproduce

  1. Get a video with white background at bottom.
  2. Share the video (public access).
  3. Enter on the public URL access.
@JoeatMJ
Copy link

JoeatMJ commented Mar 9, 2020

I have this same issue as well. Is there a why to change the playback controls to be a gray color or something other than white?

@nlights
Copy link

nlights commented Apr 20, 2020

I would also appreciate a change of color. Especially because if I don't share the video and watch it while logged in then the player has much better controls:
player

@japalie
Copy link

japalie commented Jun 23, 2020

I Solved the problem for me with editing the file videojs.js and changed the colors:
.vjs-control-bar {\n\tbackground-color: #000000 !important;\n\tcolor: #eeeeee;\n\tfont-size: 18px\n}

2020-06-23_12h01_02

It would be nice if there is a way to do it via backend and without change code.

@0xpr03
Copy link

0xpr03 commented Oct 27, 2020

Same problem here, thanks to @japalie

0xpr03 added a commit to 0xpr03/files_videoplayer that referenced this issue Oct 27, 2020
@0xpr03 0xpr03 linked a pull request Oct 27, 2020 that will close this issue
@dpalic
Copy link

dpalic commented Dec 17, 2023

similar to #41

a classical easy way solution would be to add:

  • a gradient from top black (opaque) to lower black (transparent) (or reverse direction if on bottom)
  • the gradient is about 60% of height starting
  • the font in white is in the upper part
  • so it will be always visible in any case: if background is even full white

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 a pull request may close this issue.

6 participants