deblan/side_menu

[BUG] Custom menu sorting is not applied on Mobile Screens #189

New issue

Closed

opened 2022-12-05 11:33:33 +01:00 by gnilebein · 4 comments

gnilebein commented

2022-12-05 11:33:33 +01:00

Issue

Environment

Custom menu version: 3.4.1
Nextcloud version: 25
PHP version: 8.0
Web server (Nginx, Apache2): Apache
Web browser and version (Firefox 80, Google Chrome 74, etc): Chrome & Chrome Mobile

Steps to reproduce

Set a Custom Order an open the Menu on Mobile Screen Size

Observed Results

The custom menu sorting is not applied.

Expected Results

I expect the custom menu sorting to be applied to the mobile menu as well.

## Issue ### Environment * Custom menu version: 3.4.1 * Nextcloud version: 25 * PHP version: 8.0 * Web server (Nginx, Apache2): Apache * Web browser and version (Firefox 80, Google Chrome 74, etc): Chrome & Chrome Mobile ### Steps to reproduce Set a Custom Order an open the Menu on Mobile Screen Size ### Observed Results The custom menu sorting is not applied. ### Expected Results I expect the custom menu sorting to be applied to the mobile menu as well.

gnilebein added the

label

2022-12-05 11:33:33 +01:00

deblan commented

2022-12-05 12:54:01 +01:00

Owner

On my environment, the sort is applied but apps are inversed. Do you have the same result?

On my environment, the sort is applied but apps are inversed. Do you have the same result?

gnilebein commented

2022-12-05 13:11:43 +01:00

Author

This is an example of my menu:

This is an example of my menu: ![https://i.imgur.com/KhCBzLh.png](https://i.imgur.com/KhCBzLh.png)

Ghost commented

2022-12-27 00:09:26 +01:00

Have same bug here.
It is no order which icons get in the separate menu, but there are always the same.

Have same bug here. It is no order which icons get in the separate menu, but there are always the same.

deblan added the

labels

2022-12-27 06:38:51 +01:00

deblan referenced this issue from a pull request that will close it,

2022-12-27 07:03:45 +01:00

Fix custom menu sorting is not applied on Mobile Screens #199

deblan closed this issue

2022-12-27 23:20:23 +01:00

deblan commented

2023-01-07 11:25:05 +01:00

Owner

Fixed and release in v3.5.1 ;)

Fixed and release in v3.5.1 ;)

deblan removed the

label

2023-01-07 11:25:12 +01:00

Sign in to join this conversation.

No labels

need discussion

No milestone

No project

No assignees

3 participants

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

deblan/side_menu#189

No description provided.