| Age | Commit message (Collapse) | Author |
|---|
|
Netflix doesn't reload when you change pages, so our listeners don't get
reinitialised.
This means, when you watch one video, then click the "Back to Browse"
button to go back to the /browse page, and select another video,
Immersive won't work on the newly-loaded video.
Ensure the plugin works in this case by reinitialising when the
`popstate` changes.
|
|
When the `/watch` page was opened from a Netflix `/browse` or `/title`
page, the user script wouldn't be loaded. This is because the navigation
happens via JavaScript, so Greasemonkey can't load the script.
Ensure the script is always loaded by matching all Netflix URL paths.
|
|
Don't need the old attribute value. I had added this originally to be
able to print the values to the console for development. Now it's no
longer needed.
|
|
When the credits start playing, we click on the player, giving it the
`active` class, and causing the mouse cursor to appear. The cursor
should remain hidden.
|
|
A white border appears around the frame while the credits are in
minimised mode. This is visible on the top and left sides of the frame.
Since we automatically click on the video, the white border only appears
for a second or less, but it's still visually jarring, all the more
because it shifts the position of the video.
Remove the white border, ensuring it never appears.
|
|
The age advisory ratings weren't properly hidden. Probably due to CSS
specificity. Add `!important` to ensure that the elements we want to
hide are always hidden.
|
|
|
|
|
|
|
|
Sounds better than "No Skip", and also reflects the fact that we want to
remove anything that hinders immersion while watching a video, not just
skipped credits.
|
|
Since we set `visibility: hidden` for all elements, combine them all
into a single CSS rule.
|
|
|
|
Don't use `wait_element` to hide the controls. I had done that because I
thought that the element wasn't there after clicking the "Watch Credits"
button, and I had to wait for it before trying to hide it.
Turns out the reason why the credits weren't hiding was that I just
shadowed the `controls` variable. This waiting isn't necessary, and in
fact it's undesirable, because it causes the player controls to appear
for a second before being hidden instead of not appearing at all.
|
|
Silly me, the `controls` module was getting shadowed by the `controls`
variable in `init_mutation_observer()`, which contained the DOM element.
That's why the controls weren't getting hidden.
Now that we're no longer shadowing the variable, the controls do get
properly hidden.
|
|
|
|
Old notes while I was working out elements and selectors to use.
|
|
We should insert the styles at the very start, because they hide things
that are displayed when you start watching. The credits handling always
happens at the end of the video.
Also reorder imports alphabetically.
|
|
TV series on Netflix now display a "Watch Credits" button when the
credits start rolling.
We can hide this button using CSS and the credits will continue playing
to the end, but while it's displayed, the player controls are hidden.
Automatically click the button to make it go away and reactivate the
player controls.
The tricky thing, though, is that the "Watch Credits" button doesn't
have a click handler. Instead, it listens to the "pointerdown" event, so
we have to construct one programmatically (and turn on `bubbles`,
otherwise the button doesn't react to the event) and dispatch it.
Moved `with_player` to `wait_element`, because now I need to wait for
more DOM elements than just the player element.
Add a new `controls` module since I need to hide the player controls in
both `fullscreen_credits` and `watch_credits`.
Add the `DOM` lib to `tsconfig.json` to give us DOM types.
|
|
This button appears now when the credits of TV series start playing.
|
|
Move this function to a new module as I'd like to use it for checking
the "Watch Credits" button.
Also change from a callback to a `Promise`. Need to include the
`Promise` lib via ES2015 in tsconfig to be able to build with promises.
|
|
I had initialised the `stylesheet` variable before the element existed
in the DOM.
|
|
Since it looks like everything's going to happen in the content script,
might as well make this a user script instead.
Build with Browserify in order to get a compiled JS file compatible with
browsers.
For some reason I'm currently getting an error complaining that the
`stylesheet` variable is `null`. Need to look into that.
|
|
Makes more sense to put it in the fullscreen_credits module.
|
|
That was for testing and is no longer needed.
|
|
Start to establish a bit of separation and organisation.
|
|
I'd like to split up some functionality into modules, and it seems like
using TypeScript would be a good way to achieve that.
|
