parse-dashboard icon indicating copy to clipboard operation
parse-dashboard copied to clipboard

Published 20 hours ago • verified publisher icon parse-community

demo: Scrolling issue demo [do not merge]

Open mtrezza opened this issue 2 years ago • 11 comments

New Pull Request Checklist

  • [x] I am not disclosing a vulnerability.
  • [x] I am creating this PR in reference to an issue.

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.

mtrezza avatar Jun 28 '23 19:06 mtrezza

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

parse-github-assistant[bot] avatar Jun 28 '23 19:06 parse-github-assistant[bot]

Thanks for opening this pull request!

  • 🎉 We are excited about your hands-on contribution!

parse-github-assistant[bot] avatar Jun 28 '23 19:06 parse-github-assistant[bot]

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

uffizzi-cloud[bot] avatar Jun 28 '23 19:06 uffizzi-cloud[bot]

@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

mtrezza avatar Jun 28 '23 19:06 mtrezza

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

ezgif-1-17d30cae2c

AshishBarvaliya avatar Jul 06 '23 21:07 AshishBarvaliya

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 avatar Jul 06 '23 22:07 mtrezza

@AshishBarvaliya Were you able to reproduce this?

mtrezza avatar Jul 16 '23 02:07 mtrezza

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

AshishBarvaliya avatar Jul 17 '23 23:07 AshishBarvaliya

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

mtrezza avatar Jul 19 '23 00:07 mtrezza

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

uffizzi-cloud[bot] avatar Jul 19 '23 00:07 uffizzi-cloud[bot]

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

mtrezza avatar Jul 22 '23 23:07 mtrezza