Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: Horizontal Scroll Bar in Table Widget Gone #33557

Closed
1 task done
sjr267 opened this issue May 18, 2024 · 7 comments
Closed
1 task done

[Bug]: Horizontal Scroll Bar in Table Widget Gone #33557

sjr267 opened this issue May 18, 2024 · 7 comments
Assignees
Labels
App Viewers Pod This label assigns issues to the app viewers pod Bug Something isn't working Community Reported issues reported by community members Medium Issues that frustrate users due to poor UX Needs Triaging Needs attention from maintainers to triage Production Table Widget

Comments

@sjr267
Copy link

sjr267 commented May 18, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Description

I have a table widget with more columns than will fit across the screen at once. In the past, there was a horizontal scroll bar at the bottom of the table that could be used to move the view to these columns, but this disappeared after yesterday's update. The only way I've found to scroll to these invisible columns is with the center wheel/button of the mouse.

Steps To Reproduce

In the editor, I can only get the horizontal scroll bar to appear in the table widget for a wide table if the table is the height of an integer number of rows. When I turn on cell wrapping and the rows become different heights based on the content, the horizontal scroll bar disappears because the table is no longer the height of an integer number of rows.

Public Sample App

No response

Environment

Production

Severity

Medium (Frustrating UX)

Issue video log

https://www.loom.com/share/4d80b47b1fd645ed80a7a8afe355aaee?sid=3a2c1df5-e647-4805-b952-e1029a570867

Version

Self hosted CE v1.25

@sjr267 sjr267 added Bug Something isn't working Needs Triaging Needs attention from maintainers to triage labels May 18, 2024
@Nikhil-Nandagopal Nikhil-Nandagopal added Community Reported issues reported by community members Medium Issues that frustrate users due to poor UX Production labels May 18, 2024
@github-actions github-actions bot added the App Viewers Pod This label assigns issues to the app viewers pod label May 20, 2024
@bejenaru
Copy link

on Cloud same issue

@sjr267
Copy link
Author

sjr267 commented May 21, 2024

Just discovered that this issue affects the vertical scroll bars as well. https://www.loom.com/share/5772acb273634974ac2a610f17c4233a?sid=ca1a14b4-07e0-49f7-ac05-5867d24c5032

@ame-appsmith
Copy link

Another user reported this bug.

@bejenaru
Copy link

I think the bug rise when the vertical bar is not showing.
In this case i have lots of rows and vertical bar is active resulting the horizontal one to be active also.
image
in this other case the table has only one row so the Vertical is not active. Nor the horizontal is active evenhough the table has more columns
image

@Nikhil-Nandagopal
Copy link
Contributor

@bejenaru @sjr267 we have a preview of the fix here
https://ce-33806.dp.appsmith.com

Can you try it out and let us know if it solves the problem? You will need to create a new account on the URL I shared as it's a completely separate instance

@bejenaru
Copy link

Hi @Nikhil-Nandagopal
It is working ok on this instance!

image

@Nikhil-Nandagopal
Copy link
Contributor

@bejenaru glad to hear it, the fix should be merged in soon

ankitakinger added a commit that referenced this issue May 30, 2024
…al scrollbar when needed (#33806)

## Description

Reverting the previous fix for table scrollbar and hiding vertical
scrollbar when needed, with a new fix.

Fixes [#33774](#33774)
[#33557](#33557)
[#17857](#17857)

## Automation

/ok-to-test tags="@tag.Table"

### 🔍 Cypress test results
<!-- This is an auto-generated comment: Cypress test results  -->
> [!TIP]
> 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
> Workflow run:
<https://github.com/appsmithorg/appsmith/actions/runs/9287519606>
> Commit: f4d3b4e
> Cypress dashboard url: <a
href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=9287519606&attempt=1"
target="_blank">Click here!</a>

<!-- end of auto-generated comment: Cypress test results  -->













## Communication
Should the DevRel and Marketing teams inform users about this change?
- [ ] Yes
- [ ] No
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
App Viewers Pod This label assigns issues to the app viewers pod Bug Something isn't working Community Reported issues reported by community members Medium Issues that frustrate users due to poor UX Needs Triaging Needs attention from maintainers to triage Production Table Widget
Projects
None yet
Development

No branches or pull requests

7 participants