trema-tech / moodle-theme_trema

Theme trema, a Moodle LMS plugin.
https://moodle.org/plugins/theme_trema
GNU General Public License v3.0
39 stars 19 forks source link

Mobile drawer not full screen height #191

Closed PTSdeveloper007 closed 6 months ago

PTSdeveloper007 commented 6 months ago

Prerequisites

What happened?

When Moodle on mobile view, the behavior of side menu isn't done right?

What do you think should have happened?

I expected '...'

Steps to reproduce

  1. Login as '...' (specify user role)
  2. View site as mobile
  3. Notice the behavior of side menu when clicked

Screenshots

moodle_nav

Moodle version

Moodle 4.4.0

Installed Moodle UI Language packs.

If other language(s), please specify

No response

Workaround

No response

Additional information

No response

System information

Client Device type: (e.g. Desktop, iPhone or iPad, Android phone or tablet, Other - specify) Client Operating System: (e.g. Windows, Linux, Mac OS, Android, iOS, Other - specify) Client Web Browser: (e.g. Chrome, Edge, Firefox, Safari, Safe Exam Browser, Moodle Mobile App) Server Operating System: (e.g. Windows, Linux, Mac OS, Other - specify) Server Webserver: (e.g. Apache, Nginx, IIS, Other - specify) Server PHP: (e.g. 7.4, 8.0, 8.1, 8.2, 8.3, Other - specify)

Code of Conduct

michael-milette commented 6 months ago

Hi @PTSdeveloper007 ,

Please provide additional details about what you think is not working as designed.

Best regards,

Michael Michael

PTSdeveloper007 commented 6 months ago

Hi @michael-milette,

When on mobile view, I thought the side menu will cover the logo above and the X button only comes up after a few seconds. Please confirm if this is how its supposed to be

Regards,

Peter

michael-milette commented 6 months ago

Ah! Thanks for the good suggestion. I will see what can be done about this and let you know when we have something for you to try out.

Michael

michael-milette commented 6 months ago

This issue has been resolved in the experimental dev branch. Please feel free to try it out if you can. This fix will be included in the next release of Trema.