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

Bug: {info} tags dont work in mobile app #320

Open
8 of 11 tasks
davefoord opened this issue Feb 23, 2025 · 3 comments
Open
8 of 11 tasks

Bug: {info} tags dont work in mobile app #320

davefoord opened this issue Feb 23, 2025 · 3 comments
Labels
backlog We might get to it in the future but not currently a priority

Comments

@davefoord
Copy link

Prerequisites

  • I'm reporting the issue to the correct repository.
  • I am running the latest version of this plugin (if not, please upgrade first).
  • I checked the README.md and Wiki documentation but did not find my answer.
  • I searched to make sure that this issue had not already been reported.
  • I have, or I know someone who has, the ability and willingness to test the fix on a Moodle LMS-based site.
  • I am submitting no more than one bug report per issue ticket (please do not submit multiple bugs in one issue ticket).
  • I understand that support is provided by volunteer(s).

What happened?

I have just noticed that the {info} tag doesn't appear to work when viewed on the Moodle Mobile app, which is a shame, as is a really useful feature. it may not be technically possible for it to work, but if it should and could work would be great.

What do you think should have happened?

I expected the info text to appear in a 'pop up' similar to when viewed by a browser.

Steps to reproduce

Add {info}content{/info} to a page on the site
check that it works as expected
visit the same page via the mobile app
see what happens

Screenshots

No response

Moodle version

4.1.13+ (Build: 20240927)

Installed Moodle UI Language packs.

  • English only
  • One or more other languages/regional dialects (specify below)
  • Other

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
Copy link
Owner

HI @davefoord ,

Thank you for bringing this to my attention. Unfortunately, I am not quite sure how to fix this in the mobile app. Let me know if you have any suggestions.

Best regards,

Michael

@michael-milette michael-milette added the backlog We might get to it in the future but not currently a priority label Mar 5, 2025
@davefoord
Copy link
Author

The mobile app, doesn't support the (not sure what the proper name is) pop up that you use, and not sure if it has an equivalent.

One option that does work on the mobile app, is the bootstrap details/summary mechanism that creates an accordion wit the following syntax

?

XXXXX

So a possible bodge that I can apply front end would be

{ifmobile}

?

XXXXX

{/ifmobile} {ifnotmobile}{help}

XXXXX

{/help}{ifnotmobile}

But this is a bit messy as have to apply the content twice

Could you code in the above syntax, so if not viewed through the app it behaves as current, if viewed through the app, then it works as an accordion.

If it isn't doable, then not a problem - but maybe worth adding a note against these items to warn people that they don't work on the app.

And as always - thanks again for your work on this app, which as you have probably worked out, I love.

@davefoord
Copy link
Author

And of course - the page has rendered the syntax as the accordion, which makes the above make a lot less sense. hopefully I have attached a screenshot here of the code

Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog We might get to it in the future but not currently a priority
Projects
None yet
Development

No branches or pull requests

2 participants