oneko.js icon indicating copy to clipboard operation
oneko.js copied to clipboard

move oneko with scrolling

Open rozbrajaczpoziomow opened this issue 1 year ago • 4 comments

Originally from https://github.com/Vendicated/Vencord/pull/2439#issuecomment-2111341175

rozbrajaczpoziomow avatar May 15 '24 20:05 rozbrajaczpoziomow

I don't see this as a good idea, if oneko would move as same as the page content at the same velocity and position yeah but you can't control that.

Mrgaton avatar Sep 28 '24 15:09 Mrgaton

You can do this by changing nekoEl.style.position = "fixed"; to nekoEl.style.position = "absolute"; in init() if you wanted to do it yourself, but not sure if this is seen as an issue by the creator or not because there's definitely arguments for both ways and the way it is may have been a design choice c:

st4rburn avatar Sep 28 '24 16:09 st4rburn

You can do this by changing nekoEl.style.position = "fixed"; to nekoEl.style.position = "absolute"; in init() if you wanted to do it yourself, but not sure if this is seen as an issue by the creator or not because there's definitely arguments for both ways and the way it is may have been a design choice c:

can you add it to your pull request via an option too?

Mrgaton avatar Sep 28 '24 17:09 Mrgaton

Both work and look differently, so, ignoring the fact I didn't know position absolute would do that, let's just say it was a design choice. I also don't think position absolute would work for scrolling smaller parts of the page (i.e. not the whole page, some chatbox or whatnot)

rozbrajaczpoziomow avatar Sep 28 '24 17:09 rozbrajaczpoziomow

this is implementable with CSS for users which have a preference, or users can just change the script themselves

#oneko {
    position: absolute !important;
}

Alternatively you could reimplement as an option using data attributes on the script tag like in #31

In the repo I'm gonna leave the default as it is.

adryd325 avatar Oct 29 '25 16:10 adryd325

sounds fair enough, thanks ;p

rozbrajaczpoziomow avatar Oct 29 '25 17:10 rozbrajaczpoziomow