Opened 14 years ago
Closed 13 years ago
#18654 closed defect (bug) (fixed)
Admin Bar in iOS 5 (betas 1-7) behaves unexpectedly
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 3.3 | Priority: | normal |
Severity: | normal | Version: | 3.3 |
Component: | Toolbar | Keywords: | ui-feedback has-patch needs-testing |
Focuses: | Cc: |
Description (last modified by )
Changes in MobileSafari in iOS 5 (all betas so far) are causing the Admin Bar to render unexpectedly.
Uploaded screen shots from iOS 5 Beta 7 on both iPhone and iPad. Screen shots were taken from testbp.org, which is running BuddyPress + bbPress. The admin bar pictured, while having modified menus, does not have any non-WP-core CSS that could contribute to this bug.
It appears that iOS 5 obeys fixed positioning rules differently than before. The admin bar is now properly fixed to the top regardless of the window Y position, where if memory serves in iOS 4 it used to remain at the 0px top, unfixed.
The first problem comes when zoomed. The fixed top + left positioning means you can never see the right side of the Admin Bar when zoomed in. The Admin Bar properly zooms and retains the 100% width of the browser window like I would expect it to, I just can never get to it.
The next problem appears when zoomed too. As you scroll down the bottom of the page, the Admin Bar slowly creeps upwards until it is out of view. This happens on both iPhone and iPad.
Attachments (11)
Change History (22)
#2
follow-up:
↓ 4
@
14 years ago
- Keywords ui-feedback removed
Koop is working on a new admin bar, see #18197, we can review this then again.
Also until iOS 5 isn't final we shouldn't *fix* something.
#6
@
13 years ago
- Component changed from UI to Admin Bar
- Milestone changed from Awaiting Review to 3.3
- Version set to 3.3
Given our Admin Bar and iOS prioritizing for this release, promoting to 3.3.
#7
follow-up:
↓ 9
@
13 years ago
- Owner set to georgestephanis
- Status changed from new to accepted
If noone else wants to snag this one, I'll pick it up and run with it.
I've been doing a good bit of playing around with fixed positioning in iOS before anyways.
What are we looking at for intended functionality on this? Over 90% of users are often on the latest version for iOS -- but the iPhone 1st gen, iPhone 3G, and first two generations of the iPod Touch actually cannot upgrade beyond 4.2.1
Which leads me to believe it's best to run two methods, one for iOS 5, one for iOS 4 and below. Unless the consensus is to drop support for the older versions.
#8
@
13 years ago
- Keywords has-patch needs-testing added
Very much a preliminary patch, needs testing, etc. I'll give it a good run through the muck tomorrow -- but it works well for the iPad in my brief run at it.
#9
in reply to:
↑ 7
@
13 years ago
Replying to georgestephanis:
What are we looking at for intended functionality on this? Over 90% of users are often on the latest version for iOS -- but the iPhone 1st gen, iPhone 3G, and first two generations of the iPod Touch actually cannot upgrade beyond 4.2.1
Currently the admin doesn't scale well for smart phones. There are separate apps/projects for them. So the priority would be iOS 5 on iPads and eventually other tablets (Android).
@
13 years ago
This also makes the admin menu in the admin bar work properly in RTL. And tidies up some CSS whitespace inconsistencies in wp-adminbar-rtl.dev.css
iPhone iOS 5 - No Zoom