Skip to content

Conversation

@brunopagno
Copy link
Contributor

Ticket

https://community.openproject.org/projects/communicator-stream/work_packages/69971/activity
and it might fix https://community.openproject.org/projects/communicator-stream/work_packages/69970/activity

What are you trying to accomplish?

Fix issue where the position of the placeholder would be in the wrong place when the page was scrolled.

before
imagen

Screenshots

after
imagen

What approach did you choose and why?

The placeholder text is set on a css ::before using a position: absolute. I just made sure the document has a position: relative for the component to attach to.

But I am not sure this is the proper approach.

Merge checklist

  • Added/updated tests
  • Added/updated documentation in Lookbook (patterns, previews, etc)
  • Tested major browsers (Chrome, Firefox, Edge, ...)

@brunopagno brunopagno self-assigned this Dec 23, 2025
@brunopagno brunopagno requested a review from a team December 23, 2025 10:54
@brunopagno brunopagno marked this pull request as ready for review December 23, 2025 11:20
Copy link
Member

@akabiru akabiru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@brunopagno brunopagno merged commit d4babf3 into release/17.0 Jan 5, 2026
14 checks passed
@brunopagno brunopagno deleted the fix/69971-fix-enter-text-placeholder-position branch January 5, 2026 12:35
@github-actions github-actions bot locked and limited conversation to collaborators Jan 5, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants