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

Bug:banner images dont work properly when more than one #188

Closed PTSdeveloper007 closed 6 months ago

PTSdeveloper007 commented 6 months ago

Prerequisites

What happened?

How can i edit the banner slides on the frontpage please? and when you add more than one banner image, the image moves when scrolling down. any can help with this please ? thanks

What do you think should have happened?

the banner sliders to work correctly

Steps to reproduce

  1. Login as '...' (specify user role)
  2. Go to '...'
  3. Click on '....'
  4. Scroll down to '....'
  5. See error or issue. You get it?

Screenshots

No response

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

rmady commented 6 months ago

Hello @PTSdeveloper007, Thank you for your contribution to Trema but I was unable to reproduce this bug. Please, can you send me a screenshot or a video of this issue?

Kind regards, Rodrigo Mady

PTSdeveloper007 commented 6 months ago

image

Hi, heres an image of issue. The banner slider doesnt stick to position but rather scrolls when you scroll down the page. This only happens when you add more than one image to the banner.

It works well when there is only one image in banner.

Thanks and hope to hear from you

Peter

PTSdeveloper007 commented 6 months ago

This is another issue please image

with the list buttons (attached) the padding within the buttons is uneven when hover/focus.

Hope to hear from you

Peter

PTSdeveloper007 commented 6 months ago

more here

image

michael-milette commented 6 months ago

Hi @PTSdeveloper007

Thank you for the reports. We have been able to reproduce the issue with the carousel and are working on a solutions. Hopefully, we will have a fix this soon.

As for the > not being centered, this should be fixed along with several other small issues in the experimental "dev" branch and will be included in the upcoming release shortly. Please feel free to try it out and let us know if it works for you. Note that you may need to change a theme setting (any one should do) in order for the css to recompile and take effect. This step will not be required in the released version.

In the future, we would appreciate it very much if you would kindly only include one issue per ticket as this helps us keep track of what has been fixed and what still needs attention - as indicated in the Prerequisite section of the ticket that you checked.

Best regards,

Michael Milette

rmady commented 6 months ago

Hello @PTSdeveloper007, I just created a new version with these fixes. Please, check it out and let me know if everything is ok now.

Kind regards, Rodrigo Mady

PTSdeveloper007 commented 6 months ago

This is working now thank you!