code-charity / youtube

[top~1 open YouTube & Video browser-extension] - Enrich your experience & choice! 🧰180+ options & clever features 📌set+forget📌Longest-standing(been tough). Please join🧩us👨‍👩‍👧‍👧 ..⋮ {playback|content discovery|player|extra buttons|distractions|related videos|shorts|ads|quality|codec|full tab|full screen}
http://improvedtube.com
Other
3.5k stars 527 forks source link

Hover the header on video page dont hide it entirely... #1823

Open Anoaxx opened 11 months ago

Anoaxx commented 11 months ago

...and it can be easly seen when u use theatre mode with "full window" player size option

I checked, header is dissapiring but its box its still seen. Probably cause place when box moving is aligned to the centre of height of a box not a bottom of it, and it working only like that on a video page cause i think main page works good.

ImprovedTube commented 11 months ago

with Full Height there only seems to remains some more space, than for the other two options Max width & Fit to window (yet header is invisible)

(the header also appears in grey for a moment while YouTube is loading)

Anoaxx commented 11 months ago

For me its a black strip on all the option u pointed out

gouravmpk commented 11 months ago

assign

ImprovedTube commented 11 months ago

hi! @gouravmpk thanks for caring

gouravmpk commented 11 months ago

Can I have some screen shot of the screen to understand the issue more clearly

ImprovedTube commented 11 months ago

hi @gouravmpk this is all about appeareance > player > size not reaching to the top of the window like it used to. The feature should react on both, youtube's theater mode and youtube's standard player. (And if you end up studying the feature with all options that will be great with or with out this issue, as it used to be our most used feature)


( Same applies to the options hidden on video pages & hover on video pages respectively. )


( The player size feature also used to react to the sidebar or related videos being hidden. We could fix that too or restructure or rename the the whole feature/s maybe, based on user needs )

Anoaxx commented 11 months ago

about animation. Its actually cool, if theres a way to remain it even with removing the gap it would be cool. I know you think is unnecesery, but i think its really fine ^^

ImprovedTube commented 11 months ago

So you (still) like the animation? because you don't need it every time? Just assuming fading in might become the first one listed. To save user's attention(=time). - Won't mind hoarding both of course. (Renaming hover on video page (slide animation) ( Dropdown options space is unlimited)

Anoaxx commented 11 months ago

What do you mean? I would like gap to dissapear, but showing and hiding animation has nothing to do with gap

ImprovedTube commented 11 months ago

Hope the (optional!) complexity didn't overwhelm you @gouravmpk!

if theres a way to remain it

animation has nothing to do with gap

still just thought fading in from transparent is more calm and might take a line of code less as said, hence expecting both options.

While one issue at a time is enough, one cant only care for half a feature. It might be efficient to try to consider the road-net ahead and one can end up fixing two things at once just with the same effort and commenting/planing/helping a 3rd one.

Anoaxx commented 11 months ago

Oh i find something interesting that is cool. I u have header hidden on video page, u can still went into finding it clicking "go to search box" shortcut. Just addition, it would be cool if you would add explaination in that option. Cause im using your extension for a looong time and finding this only now xD

Oh and u cant do that on non-video pages somehow

ImprovedTube commented 10 months ago

@Anoaxx

explanation

let's add a tooltip. I think you nearly earned a PR 🤣

cant .. on non-video pages

can you check for error? do other features als stop working on non-video pages?

Anoaxx commented 8 months ago

@ImprovedTube After new update, the top black space is nowhere to be found, but theres another issue... When im changing my uption to "hover" my entire page is slighty moved up, and even usin "full height" option is leaving ha bar with title on a bottom. I need to investigate more tho, but u can try to reproduce it. My settighs are here: improvedtube (5).zip

Anoaxx commented 8 months ago

After some more tests: I reinstalled extension, and its still making player smaller when u use Hover/hover on video page and somehow with static option.

i changed theme to show it better image

Anoaxx commented 8 months ago

oh and after ecent update scrollbars cant be hidden. Somehow any of extensions i got to do that, stopped working. Illl made new issue.

ImprovedTube commented 8 months ago

right, we need to tune one more variable #2027

Anoaxx commented 8 months ago

Emmm... top black bar™️ returned @ImprovedTube

AyushBarai commented 6 months ago

is this issue is closed ?

ImprovedTube commented 6 months ago

hi! @AyushBarai no, just tried the one variable and undid.

Anoaxx commented 5 months ago

image

@ImprovedTube After new update on Chrome Webstore for beta(4.928) header is fully hidden, but using its still causing to player not being on full height, even if player size is set to full height.

Anoaxx commented 5 months ago

PS. Its a little bit smaller when u scale page to be smaller 🤷

Anoaxx commented 5 months ago

I can suggest to make additional option that u can turn on, ane then u can choose the header behaviour seperatly for home page and video page.

ImprovedTube commented 5 months ago

which two will you chose?

Anoaxx commented 5 months ago

What do you mean exactly?