search

parse-community / parse-dashboard

A dashboard for managing Parse Server
https://parseplatform.org
Other
3.74k stars 1.39k forks source link

demo: Scrolling issue demo [do not merge] #2486

Closed mtrezza closed 4 months ago

mtrezza commented 1 year ago

New Pull Request Checklist

Issue Description

Closes: https://github.com/parse-community/parse-dashboard/issues/2045#issuecomment-1609537846

Approach

This is just a demo for the scrolling issue that has been introduced with https://github.com/parse-community/parse-dashboard/pull/2457. See https://github.com/parse-community/parse-dashboard/issues/2045#issuecomment-1609537846.

This PR is not supposed to be merged.

parse-github-assistant[bot] commented 1 year ago

I will reformat the title to use the proper commit message syntax.

parse-github-assistant[bot] commented 1 year ago

Thanks for opening this pull request!

uffizzi-cloud[bot] commented 1 year ago

Uffizzi Ephemeral Environment deployment-29690

:watch: Updated Jun 28, 2023, 19:21 UTC

:cloud: https://app.uffizzi.com/github.com/parse-community/parse-dashboard/pull/2486

:page_facing_up: View Application Logs etc.

What is Uffizzi? Learn more

mtrezza commented 1 year ago

@AshishBarvaliya here are the instructions to replicate the scroll issue described in https://github.com/parse-community/parse-dashboard/issues/2045#issuecomment-1609537846:

  1. Open the preview environment
  2. Open class "MyClass"
  3. Scroll to the very right and very bottom corner
  4. Open class "_User"
  5. Notice that the scroll position is already strange, there is a lot of blank space to scroll around, even thought there are 0 rows
  6. Click on "Add row" button in the top bar
  7. Scroll to the top (but keep scrolled to the very right)
  8. Scroll to the left and see how the headers move independently from the row fields

https://github.com/parse-community/parse-dashboard/assets/5673677/06a0ebf1-eec4-421a-977e-95f0b9d04d88

AshishBarvaliya commented 1 year ago

@mtrezza I don't know what I am missing here but I am still not able to reproduce the issue.

ezgif-1-17d30cae2c

mtrezza commented 1 year ago

Looks like a browser compatibility issue. I'm using Safari Version 16.4 (18615.1.26.110.1) and I'm able to reproduce the issue with the preview environment and following the same steps in your video. What browser are you using?

mtrezza commented 1 year ago

@AshishBarvaliya Were you able to reproduce this?

AshishBarvaliya commented 1 year ago

Seems like width: calc(100% - 300px); was causing the issue in Safari. Please try with this alternative suggestions.

mtrezza commented 1 year ago

Done, lets see how it goes when the preview env is recreated

uffizzi-cloud[bot] commented 1 year ago

Uffizzi Ephemeral Environment deployment-31232

:watch: Updated Jul 19, 2023, 00:16 UTC

:cloud: https://app.uffizzi.com/github.com/parse-community/parse-dashboard/pull/2486

:page_facing_up: View Application Logs etc.

What is Uffizzi? Learn more

mtrezza commented 1 year ago

The issue now is that when scrolling into the bottom right corner, and then changing to a different class with many rows, the data browser does not start at the top most (first) row, but at a lower row.

https://github.com/parse-community/parse-dashboard/assets/5673677/350478f2-f27f-4311-bebc-9c47538b7376