Some comments : * Using $context.setLinksQueryString("skin=${xwiki.skin}") is not really acceptable. It breaks when you have AJAX. The skin could rather be determined server-side based on the cookie you set. * There are some ios-specific naming ("iosdisplaypanel, div class="ios", etc.") * The UA detection could probably be improved * You have some inline (onclick="blah") and un-namespaced (see showQuickLinks in header.vm) JS, which are respectively not respecting web standards and our dev standards (http://dev.xwiki.org/xwiki/bin/view/Community/DevelopmentPractices#HJavaScri...). More than those implementation details, I wonder if this is the right approach. Since the differences with colibri are not very big, I personally would prefer to see them implemented using responsive design right in colibri. Jerome On Tue, Feb 21, 2012 at 8:30 PM, Ludovic Dubost <[email protected]> wrote:
Any comments on the code ? What would need to be done to be able to bring this into the platform ?
Ludovic
2012/2/17 Ludovic Dubost <[email protected]>
Hi,
I've been working a while ago on a Mobile Skin which principle is the following:
1/ Light UI adapted to mobile devices, both phones and tablet (with a particular attention to tablets), giving most of the screen real estate to the page content 2/ Support all Javascript normally supported in the XWiki skin in order for any code inside pages that would use this Javascript to still work 3/ Automatic detection of mobile devices to make them switch to the mobile skin 4/ Ability to switch from mobile to normal skin 5/ compatibility with color themes
I believe this approach is the right one, as XWiki instances do have significant Javascript and in order to make sure these pages work, we need to be able to accept this javascript. Of course some other improvements could be done, for example to have a mobile friendly dashboard or livetable.
Now this work has had a few iterations with particularly some relooking work from Caty. The work is published here:
http://extensions.xwiki.org/xwiki/bin/view/Extension/SimpleMobileSkin
And the code is available here in both XAR and File-System version (for XEM):
https://github.com/ldubost/xwiki-mobileskin
The skin can be tested on the incubator (although there is not the automated switch)
http://incubator.myxwiki.org/xwiki/bin/view/Improvements/34Proposal?skin=XWi...
I've also worked on integrating this in the platform. The commit is available in a fork on github:
https://github.com/ldubost/xwiki-platform/commit/064baeb017b35e08a5260293814...
I'd like some comments and discussion so that we can bring this into the platform.
Ludovic
-- Ludovic Dubost Founder and CEO Blog: http://blog.ludovic.org/ XWiki: http://www.xwiki.com Skype: ldubost GTalk: ldubost
-- Ludovic Dubost Founder and CEO Blog: http://blog.ludovic.org/ XWiki: http://www.xwiki.com Skype: ldubost GTalk: ldubost _______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
-- Jérôme Velociter Winesquare http://www.winesquare.net/