Open-event-frontend: Organizer Session Page: Dropdown menu results in internal scroll bar

Created on 14 Oct 2020  路  15Comments  路  Source: fossasia/open-event-frontend

When an organizer clicks on status pending, accepting etc. in the last row of a page suddenly an internal scroll bar shows up and the dropdown menu appears below other page elements.

Expected: No internal scroll bar. Elements should be on top of below elements.

Screenshot from 2020-10-14 08-02-37

bug codeheat

All 15 comments

Expected: No internal scroll bar. Elements should be on top of below elements.

It is showing in that way when it is as the end of wondow not at end of table. Because semantic ui don't work in that way. Because for each cell the dropdown behaviour would be compressed then and be limited to that cell. it shows upside when it is at the end of wondow.
Screenshot from 2020-10-14 16-11-18

Making overflow: visible for div id="ember1057" solves the scroll bar issue. But my question is can we do that as the top bar of the table (thead) id="ember1058" is position: static and scroll down with the display?
image

When an organizer clicks on status pending, accepting etc. in the last row of a page suddenly an internal scroll bar shows up and the dropdown menu appears below other page elements.

Expected: No internal scroll bar. Elements should be on top of below elements.

Screenshot from 2020-10-14 08-02-37

sir, can you please share the link to the page or location of the file in the codebase.

@mariobehling There are 2 possible solutions:

  1. Expand the table from below so the dropdown doesn't get cut
  2. Instead of dropdown, a modal popup is shown

@iamareebjamal I think a pop up is too big of a thing. I saw the screenshot above of @abhpd which looked good, but if that has other issues just extending seems fine, do you think?

image

  • That'll overflow the table contents, so let's go with table expansion

I am going on table expansion.

I implemented table expansion.

The current solution works. But the logic is too complicated. Reopening in hope of a codeheat participant finding a better solution

@iamareebjamal I think the issue is not yet resolved.
This is what I am getting same behavious as was before PR was merged
Screenshot from 2020-12-09 19-35-13

I had to revert the PR because it was caausing other issues

@iamareebjamal @mariobehling I might have a better solution, can you assign it to me? Thank you.

submit a PR

@iamareebjamal @mariobehling I might have a better solution, can you assign it to me? Thank you.

Are you still working on it?

@iamareebjamal @mariobehling I might have a better solution, can you assign it to me? Thank you.

Are you still working on it?

No.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

mariobehling picture mariobehling  路  3Comments

hpdang picture hpdang  路  5Comments

harsh-jindal picture harsh-jindal  路  5Comments

mariobehling picture mariobehling  路  4Comments

codedsun picture codedsun  路  4Comments