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.

Sign up for GitHub

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

Jump to bottom

Mobile view: editing ACLs not possible #2020

Closed
StefanAbel-OTOBO opened this issue Nov 30, 2022 · 2 comments
Closed

Mobile view: editing ACLs not possible #2020

StefanAbel-OTOBO opened this issue Nov 30, 2022 · 2 comments
Labels
style Beautify please

Comments

@StefanAbel-OTOBO
Copy link
Contributor

When you are in mobile view and want to edit an existing ACL in Action=AdminACL;Subaction=ACLEdit , there is no "Save"-Button. You neither find it below the ACL:
image

... nor can you find it in the menu on the right:
image

This means that currently it is not possible to edit any ACL on a mobile device.
@StefanAbel-OTOBO StefanAbel-OTOBO added the style Beautify please label Nov 30, 2022
@stefanhaerter
Copy link
Contributor

I can confirm this. The saving widget appears at a screen width of 1025px and beyond, which - as far as my tests showed - is caused by the widget having the ScreenXL class instead of the Expanded class like the other widgets. Before changing this, I would like to discuss possible intentions for setting this class and potential side effects.

stefanhaerter added a commit that referenced this issue Dec 1, 2022
@stefanhaerter
Issue #2020: Setting class of saving widget similar to the other widg…
8429cb3 
…ets on the page
stefanhaerter added a commit that referenced this issue Dec 1, 2022
@stefanhaerter
Issue #2020: Setting class of saving widget similar to the other widg…
ae304f8 
…ets on the page
svenoe pushed a commit that referenced this issue Dec 1, 2022
@stefanhaerter @svenoe
Issue #2020: Setting class of saving widget similar to the other widg…
4dedcfa 
…ets on the page
@stefanhaerter
Copy link
Contributor

PR merged, closing this as complete.

@stefanhaerter stefanhaerter mentioned this issue Dec 1, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
style Beautify please
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@stefanhaerter @StefanAbel-OTOBO