Remove input scroll from position

[Link2Twenty]

What type of PR is this? (check all applicable)

• Refactor
• Feature
• Bug Fix
• Optimization
• Documentation Update

Description

When an input has a scroll we remove its position from the overall position putting the mention popover in the right place

Related Tickets & Documents

closes #13513

QA Instructions, Screenshots, Recordings

UI accessibility concerns?

N/A

Added tests?

• Yes
• No, and this is why: existing tests will still work
• I need help with writing tests

[optional] Are there any post deployment tasks we need to perform?

N/A

[optional] What gif best describes this PR or how it makes you feel?

As a side note I finally installed a local copy of forem rather than just winging it 😅

[github-actions]

Thank you for opening this PR! We appreciate you!

For all pull requests coming from third-party forks we will need to
review the PR before we can process it through our CI pipelines.

A Forem Team member will review this contribution and get back to
you as soon as possible!

[Link2Twenty]

As another aside ddtrace 0.48.0 doesn't work, I had to force downgrade to 0.47.0 to get the forem setup to work

[rhymes]

@Link2Twenty sorry about that, we eventually reverted to 0.47 #13570

[Link2Twenty]

@Link2Twenty sorry about that, we eventually reverted to 0.47 #13570

It's alright, most of my debugging time was spent trying to work out what I did wrong, once I realised it wasn't my fault it was an easy fix 😅

[Link2Twenty]

I don't think the Travis fail is me, can someone verify?

[rhymes]

Nope, not related to your code, @Link2Twenty. I re-ran Travis!

[rhymes]

LGTM!

[aitchiss]

Nice one! Thanks for taking this on - looks great to me ✨

[cmgorton]

Thanks for taking the time to work on this @Link2Twenty !