Request to add code to solve issue layout out of splitview area. about rsselectionmenu HOT 9 CLOSED

I edited and added the screenshot.

I only added this line of code into your RSSelectionMenuController, and it fixed my problem.

self.backgroundView.frame = self.view.bounds

from rsselectionmenu.

@praditkan Could you please add a screenshot, what's the issue?
And probably the code how you're presenting the menu?

from rsselectionmenu.

@praditkan ideally any popup (formSheet) background should cover entire window width (master + detail), not only detail view.
You can take reference from iPad Settings app or alert presented on the iPad.

If we allow the user to select the item from the master view, while any popup is on the screen, then it's a bad UX because the user is expected to take any action in the popup or dismiss it.

from rsselectionmenu.

That is true in a sense of design best practice and preference.

However, your code still allow us to use this way.
Then this is still a defect, isn't it?
It displays off the screen.

I just hope this would make it more robust to use.
I could be wrong sorry.
I'm a new swift developer anyway.

from rsselectionmenu.

Maybe you can present from splitViewController and then check?
menu.show(type: <type here>, from: self.splitViewController)

from rsselectionmenu.

@praditkan Were you able to resolve this?

from rsselectionmenu.

Ah this one work!!!

menu.show(type: <type here>, from: self.splitViewController)

Thank you so much.

from rsselectionmenu.

I found another issue.

The same project I did change self.splitViewController as you advised for those detail view.

It happens again this time within the view that is presented modally.

I tried this but it doesn't work.

menu.show(type: <type here>, from: self.presentingViewController!)

from rsselectionmenu.

@praditkan why did you use from: self.presentingViewController! here?
I believe it's not the issue in the library. Maybe you can try presenting from another view controller or application window.

Closing this issue for now:)

from rsselectionmenu.