My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 2857: Mobile stylesheet contains elements of Slim Shaded skin
2 people starred this issue and may be notified of changes. Back to list
 
Project Member Reported by sarken, Jan 25, 2012
What archive revision are you testing on? (See the latest release notes in
http://archiveofourown.org/admin_posts, for example 0.8.4.4).
otwarchive v0.8.10.22

If appropriate, enter the URL of a page where the problem can be seen:
Any page viewed on mobile

What steps will reproduce the problem?
1. View a page on mobile

What is the expected output? What do you see instead?
When viewed on a mobile device, the site has some colors and shadows in the header that don't fit with the normal design.

Please use labels and text to provide additional information.
There's a comment in the mobile stylesheet (http://archiveofourown.org/stylesheets/site/2.0/25-role-handheld.css): "pasted in the Slim Shaded look temporarily to try to fix for iphone--needs test." Sure enough, if you apply the Slim Shaded skin to the non-mobile site (http://archiveofourown.org/skins/930/preview), you see the elements that look so out of place on mobile.

Jan 25, 2012
Project Member #1 sarken
Totally forgot my screen shots. The first is what My Home looks like with a header link clicked. The second is what the works page looks like when you're logged out.
mobile_slim_myhome_logged_in.png
127 KB   View   Download
mobile_slim_works_logged_out.png
150 KB   View   Download
Jan 31, 2012
Project Member #2 eni...@gmail.com
(No comment was entered for this change.)
Labels: Milestone-Internal0.8
Jan 31, 2012
Project Member #3 eni...@gmail.com
(No comment was entered for this change.)
Labels: Roadmap-Skins
Feb 2, 2012
Project Member #4 sarken
(No comment was entered for this change.)
Status: Started
Feb 3, 2012
Project Member #5 sarken
https://github.com/otwcode/otwarchive/pull/464

Testers, all but one of these changes are in the header. The non-header change involves restoring the button-like appearance of the navigation actions throughout the site. If you have something other than an iPhone 4, your input is especially welcome.
new_mobile.PNG
214 KB   View   Download
Status: SubmittedPullRequest
Feb 3, 2012
Project Member #6 shal...@gmail.com
(No comment was entered for this change.)
Status: FixedAndCommitted
Feb 12, 2012
Project Member #7 cesy.a...@gmail.com
(No comment was entered for this change.)
Status: DeployedToTest
Feb 15, 2012
Project Member #8 elz...@gmail.com
(No comment was entered for this change.)
Labels: Release-0.8.11
Feb 25, 2012
Project Member #9 cesy.a...@gmail.com
Works index is fixed on Android (HTC Desire S) - the black to red header nav change shown in sarken's screenshots was reproduced by comparing Beta and Test.
Feb 25, 2012
Project Member #10 highlander.ii@gmail.com
I have an iPhone 3GS - if there's anything specifically you want me to check, let me know.  I have a couple screenshots to add once they reach my email from my phone - can't upload photos to g-code from my iPhone.
Feb 26, 2012
Project Member #11 irene.ot...@gmail.com
Ipod touch 4th gen. Both logged in and logged out, header looks the same as in screen shots from commenter 1 (white text on grey header).
Feb 26, 2012
Project Member #12 highlander.ii@gmail.com
here is my screenshot from test archive - default skin 'logged in'
defaultskin-test.PNG
59.9 KB   View   Download
Feb 27, 2012
Project Member #13 sarken
Irene, it sounds like a cache problem. You might need to go to Settings -> Safari -> Clear History and/or Clear Cookies and Data.
Mar 1, 2012
Project Member #14 myowncon...@gmail.com
Talked to elz, calling this verified. Not deploy blocking. Not testing on a BlackBerry but if we get any bugs logged we will look at it. :)
Status: VerifiedOnTest
Mar 1, 2012
Project Member #15 elz...@gmail.com
(No comment was entered for this change.)
Status: DeployedToBeta
Sign in to add a comment

Powered by Google Project Hosting