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

3 dots are missing on Groups page on mobile #6126

Open
amroland opened this issue Nov 16, 2022 · 3 comments
Open

3 dots are missing on Groups page on mobile #6126

amroland opened this issue Nov 16, 2022 · 3 comments
Milestone

Comments

@amroland
Copy link

There is no 3 dots on mobile on Groups page.

Steps to Reproduce

  1. Open your WP admin page on mobile.
  2. Go to Sensei LMS » Groups menu.
  3. You can see that there is no 3 dots at the end of the row of your groups (if you have any).

What I Expected

There should be 3 dots to use on mobile like on PC.

What Happened Instead

There are no 3 dots on mobile.

PHP / WordPress / Sensei LMS version

WordPress 6.1
PHP 7.4.26
Sensei LMS Version: 4.6.4 | Sensei Pro Version: 1.6.2

Screenshot / Video

Képernyőfotó 2022-11-16 - 12 36 47

Context / Source

@amroland amroland added [Status] Needs Triage Issue needs to be triaged [Type] Bug labels Nov 16, 2022
@mikeyarce
Copy link
Member

I'm not sure this is a bug but more a design choice on our part. If you click through to the specific group, can you perform all the actions?

My concern for showing a context menu like this on Mobile would be that it might not display properly on a small screen.

@amroland
Copy link
Author

Hi @mikeyarce,

No, if you click through you can't perform all action which is included in 3 dots menu. That is a deeper submenu. You can add only members. You can't rename or delete group, for example.
(I don't want to offend anyone, I'm just asking if you, in development, can't check this error on any mobile? You should see what I can see.)

And 3 dots appear on mobile on Students page, too. Although it has issue, too: #6125

There should be a solution to use those actions on mobile, too.

@mikeyarce
Copy link
Member

Hi @amroland,

You can't rename or delete group, for example.

Thanks for the info! I could definitely see that the menu was missing, I just wanted to understand better what actions couldn't be done that you were wanting to do on mobile.

@mikeyarce mikeyarce removed the [Status] Needs Triage Issue needs to be triaged label Nov 18, 2022
@donnapep donnapep added this to the 4.24.4 milestone Aug 22, 2024
@donnapep donnapep modified the milestones: 4.24.4, 4.24.5 Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants