Skip to content

Conversation

@prakhar0608shrma
Copy link

Summary

Fixes an issue where an extra scrollbar appears in the “Report error” flow due to overflow behavior in a shared component, resulting in a confusing user experience.

The update ensures scrolling is handled correctly and avoids nested or duplicate scrollbars.

Changes

  • Adjusted overflow behavior in TechnicalText.vue to prevent nested scrolling
  • Improved scrollbar handling for views that embed this component, including the Report error flow

Testing

  • Verified that the “Report error” flow no longer shows an extra scrollbar
  • Confirmed no regressions in scrolling behavior for views using TechnicalText.vue

Issue

Closes #5922

Screenshots

N/A

- Added overflow-y: auto to textarea to handle scrolling internally
- Prevents third scrollbar from appearing in modal

Fixes learningequality#5922
@learning-equality-bot
Copy link

Season's greetings! 👋

We'd like to thank everyone for another year of fruitful collaborations, engaging discussions, and for the continued support of our work. Learning Equality will be on holidays from December 22 to January 5. We look forward to much more in the new year and wish you a very happy holiday season!

Are you preparing for Google Summer of Code? See our GSoC guidelines.

@learning-equality-bot
Copy link

👋 Thanks for contributing!

We will assign a reviewer within the next two weeks. In the meantime, please ensure that:

  • You ran pre-commit locally
  • All issue requirements are satisfied
  • The contribution is aligned with our Contributing guidelines. Pay extra attention to Using generative AI. Pull requests that don't follow the guidelines will be closed.

We'll be in touch! 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Extra scrollbar in report error modal

1 participant