#2860 closed defect (fixed)

docs.rtems.org does not work on iPhones or iPads

Reported by: Chris Johns Owned by: Chris Johns
Priority: highest Milestone: Indefinite
Component: doc Version:
Severity: major Keywords:
Cc: Blocked By:


The accordion box collapse does not work on iPhone and iPad.

Attachments (1)

0001-docs.rtems.org-fix-website-in-iPhones-and-iPads.patch (1.6 KB) - added by Marçal Comajoan Cara on 12/10/18 at 19:18:06.
Patch to fix the bug

Download all attachments as: .zip

Change History (11)

comment:1 Changed on 05/11/17 at 07:31:02 by Sebastian Huber


comment:2 Changed on 08/13/17 at 23:39:11 by Chris Johns

Version: 4.114.12

comment:3 Changed on 10/10/17 at 06:06:29 by Sebastian Huber

Component: Documentationdoc

comment:4 Changed on 10/11/17 at 23:12:34 by Chris Johns

Status: newaccepted

comment:5 Changed on 11/09/17 at 06:27:14 by Sebastian Huber


Milestone renamed

comment:6 Changed on 10/14/18 at 00:33:55 by Joel Sherrill

Milestone: 5.1Indefinite
Version: 5

comment:7 Changed on 12/06/18 at 15:52:25 by Marçal Comajoan Cara

I downloaded the https://docs.rtems.org website with a website downloader and made a local web server to be able to test the changes I make from the computer in an iPad. I used an iPad 2 from 2011 with iOS 9.3.5 to test it so it should work on newer iPhones and iPads too.

Basically what happens is that the code is correct (this is why it works in every other web browser), but relies only on the data-target and data-toggle attributes, which for iPad's and iPhone's Safari web browser don't work very well. Other iOS web browsers (Chrome, Firefox, etc. for iOS) are probably also affected by this bug because Apple only permits to use the Safari browser engine in iOS applications.
Just by adding some CSS (cursor: pointer;) to the element works as expected. This change also shows a pointer cursor when hovering over the accordion you have to click when using a computer which makes it more intuitive instead of just an arrow.

I think that the source code of https://docs.rtems.org is not in any public repository so I can't make a patch, but the code to add is really short and simple. Just add this code at the end of https://docs.rtems.org/static/rtems/css/rtems.css:

[data-toggle~="collapse"] {
    cursor: pointer;

The code adds the required CSS to all elements which have the attribute data-toggle="collapse", so it fixes every element and everything works as expected.

comment:8 Changed on 12/09/18 at 23:44:12 by Chris Johns

This is awesome.

The docs web site is held in this repo https://git.rtems.org/chrisj/rtems-admin.git/. If you could please make a patch against this repo I can apply it and push it.

Thank you for taking a look and providing a solution.

comment:9 Changed on 12/10/18 at 19:17:15 by Marçal Comajoan Cara

Ok, patch done! (attached and sent to devel@)

Changed on 12/10/18 at 19:18:06 by Marçal Comajoan Cara

Patch to fix the bug

comment:10 Changed on 12/13/18 at 22:06:58 by Chris Johns

Resolution: fixed
Status: acceptedclosed

Pushed the patch to the repo and uploaded the change to docs.rtems.org so it is now live. I have tested the change on IOS and all is working. Thank you for looking into this and fixing it.

Note: See TracTickets for help on using tickets.