mirror of
https://github.com/hoppscotch/hoppscotch.git
synced 2026-04-25 16:55:59 +03:00
[GH-ISSUE #266] [BUG] [UI] [Mobile] Get results not scrollable. #90
Labels
No labels
CodeDay
a11y
browser limited
bug
bug fix
cli
core
critical
design
desktop
discussion
docker
documentation
duplicate
enterprise
feature
feature
fosshack
future
good first issue
hacktoberfest
help wanted
i18n
invalid
major
minor
need information
need testing
not applicable to hoppscotch
not reproducible
pull-request
question
refactor
resolved
sandbox
self-host
spam
stale
testmu
wip
wont fix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/hoppscotch#90
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @AtomicMaya on GitHub (Nov 5, 2019).
Original GitHub issue: https://github.com/hoppscotch/hoppscotch/issues/266
Originally assigned to: @hosseinnedaee on GitHub.
Describe the bug
When GETting a larger quantity of data, the Results area does not allow scrolling (the page gets scrolled instead).
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Scroll should be deferred to Text Area instead of page.
Screenshots

Smartphone (please complete the following information):
@hosseinnedaee commented on GitHub (Nov 8, 2019):
I checked it on the same device and it works fine for me (Chrome & Opera).
@liyasthomas commented on GitHub (Nov 8, 2019):
Normal responses with LOC < 100 works fine.
Huge payload breaks HighlightJS render. Issue will persist until syntax highlighting is migrated to ace.
@hosseinnedaee commented on GitHub (Nov 8, 2019):
I will work on it.
@liyasthomas commented on GitHub (Nov 8, 2019):
Currently, response is syntax highlighted in
<pre><code>tags for highlightJS implementation. Remove hljs and implement alternative (such as ace) that supports textarea highlighting@hosseinnedaee commented on GitHub (Nov 8, 2019):
Okay
@AtomicMaya commented on GitHub (Nov 12, 2019):
Yup, This was fixed !