The Official "Is Something Broken?" Thread

No, that’s the “timeline” moving. It should move to avoid covering the BP/Flag/Reply buttons on the post nearest the bottom of the screen.

I can’t reproduce the “flicker” - what browser and OS are you using? Researching on their forums.

Nevermind, got it to reproduce. Had to slow down my scrolling. Looking into it.

@physical would the better trash placement be right-aligned next to the upload button? Both of those would have confirmation prompts at least.

2 Likes

@villain I can also reproduce this, looks like our CSS tweaks around the post count box are fucking up at the end of threads now. I’m not sure how much we need to do to revert that; iirc our main override here was to use the post count box from mobile on desktop as well so that we could have 1000px width posts instead of a floating sidebar like other discord sites, and idk if the override contains old code in it

4 Likes

thanks for looking into it

(i mentioned it because it just started happening so i assumed it was the result of a recent change)

Also have this on Win10 Chrome.

also pinging @toups if they have availability so I’m not overburdening @villain

1 Like

Did somebody do this?

image

I highly recommend against making modifications to active components on active themes before you know they work.

I don’t think anyone touches the CSS except for like, 90% you and 10% toups, I think this is probably a weird interaction from our overrides + an upstream upgrade

Yeah this block jumps from the bottom right outside of the post area to top right within the post area (and hence blocking parts of the posts) near the bottom as of tonight.

Just to confirm, wasn’t me and no one else has changed themes per the logs.

is this on mobile or desktop? doesn’t seem to happen for me on mobile.

d   s   t   p
 
  e   k   o

5 Likes

happens to me and others too

yeah i’ve also been getting it on MacOS Firefox

what a mystery

yes, it’s a desktop issue with the way we use the post count box from mobile. I suspect the override needs to be updated.

I’ll take a look at it today

1 Like

okay so best I can tell this doesn’t seem to be a result of any css overrides that we’ve done. the site javascript is adding a class to the post count box when the scroll bar reaches a certain threshhold, and that’s messing up the positioning. this might actually be a discourse bug, unless we have some custom javascript added somewhere that does this?

1 Like

changing positioning is normal behaviour but I think we’ve moved something by a few px that fucks it up, it might just need an offset

that makes sense. let me dig through our add-ons then.

Should it move underneath the topic list or just underneath the header? the code seems to want it to move to the header… is that what our extension aims to override?

AFAIK the extension overrides a completely different desktop timeline UI element with more (too much?) detail.