[GH-ISSUE #2019] [Bug] Unable to scroll/access all the menu-items of slash menu when app-window size is smaller #799

Closed
opened 2026-03-23 20:41:14 +00:00 by mirror · 4 comments
Owner

Originally created by @Samiksha-Garg on GitHub (Mar 19, 2023).
Original GitHub issue: https://github.com/AppFlowy-IO/AppFlowy/issues/2019

Originally assigned to: @Samiksha-Garg on GitHub.

Bug Description

On small window sizes, the width of the slash menu becomes is greater and the menu overflows the right boundary. With no horizontal scroll available, the user is unable to interact with all the menu items in such case.

How to Reproduce

Steps to reproduce the behaviour:

  1. Make the app-window size as small as possible (or smaller than the expected dimensions of the slash menu)
  2. Type "/" to invoke the slash menu command.

Expected Behavior

It would be better if user could just scroll over the overflowing items.

Operating System

MacOS (arm64)

AppFlowy Version(s)

0.1.0

Screenshots

Screenshot 2023-03-19 at 9 44 13 AM

In the screenshot attached I am unable to access/see callout menu item.

Additional Context

No response

Originally created by @Samiksha-Garg on GitHub (Mar 19, 2023). Original GitHub issue: https://github.com/AppFlowy-IO/AppFlowy/issues/2019 Originally assigned to: @Samiksha-Garg on GitHub. ### Bug Description On small window sizes, the width of the slash menu becomes is greater and the menu overflows the right boundary. With no horizontal scroll available, the user is unable to interact with all the menu items in such case. ### How to Reproduce Steps to reproduce the behaviour: 1) Make the app-window size as small as possible (or smaller than the expected dimensions of the slash menu) 2) Type "/" to invoke the slash menu command. ### Expected Behavior It would be better if user could just scroll over the overflowing items. ### Operating System MacOS (arm64) ### AppFlowy Version(s) 0.1.0 ### Screenshots <img width="814" alt="Screenshot 2023-03-19 at 9 44 13 AM" src="https://user-images.githubusercontent.com/79906086/226153762-81afee99-aa64-4794-9065-5b3486ddc0e4.png"> In the screenshot attached I am unable to access/see callout menu item. ### Additional Context _No response_
Author
Owner

@Samiksha-Garg commented on GitHub (Mar 19, 2023):

Hello! I have found a fix around this small bug! Can I take this up?

<!-- gh-comment-id:1475100261 --> @Samiksha-Garg commented on GitHub (Mar 19, 2023): Hello! I have found a fix around this small bug! Can I take this up?
Author
Owner

@LucasXu0 commented on GitHub (Mar 20, 2023):

Hey, @Samiksha-Garg. Sure. How do you plan to fix it?

<!-- gh-comment-id:1475657795 --> @LucasXu0 commented on GitHub (Mar 20, 2023): Hey, @Samiksha-Garg. Sure. How do you plan to fix it?
Author
Owner

@Samiksha-Garg commented on GitHub (Mar 20, 2023):

Hey @LucasXu0, So the selection menu is wrapped inside a positioned widget, and in order to make it horizontally scrollable it was necessary to have both left and right position arguments in place. Right position argument was missing so added that, and wrapped the child in single child scrolling widget. Selection menu scrolls only in case of overflow, otherwise it takes as much as width as required. I will raise a PR for it soon.

<!-- gh-comment-id:1475768928 --> @Samiksha-Garg commented on GitHub (Mar 20, 2023): Hey @LucasXu0, So the selection menu is wrapped inside a positioned widget, and in order to make it horizontally scrollable it was necessary to have both left and right position arguments in place. Right position argument was missing so added that, and wrapped the child in single child scrolling widget. Selection menu scrolls only in case of overflow, otherwise it takes as much as width as required. I will raise a PR for it soon.
Author
Owner

@richardshiue commented on GitHub (Mar 20, 2023):

Hey there @Samiksha-Garg, could the items be re-arranged in less columns instead of it becoming scrollable in that situation? I think it would look better.

<!-- gh-comment-id:1476120979 --> @richardshiue commented on GitHub (Mar 20, 2023): Hey there @Samiksha-Garg, could the items be re-arranged in less columns instead of it becoming scrollable in that situation? I think it would look better.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
AppFlowy-IO/AppFlowy#799
No description provided.