Move iPhone CSS from wp-login.php to wp-admin.css

[SergeyBiryukov]

There are some iPhone-specific CSS rules in wp-login.php:
​http://core.trac.wordpress.org/browser/tags/3.3/wp-login.php#L67

wp-admin.css seems to be a more appropriate place for them.

Related: #10023, #19331

[SergeyBiryukov]

19673.patch​ uses iphone class.

I've noticed that Toolbar uses mobile (doesn't seem to be used anywhere else in core though):
​http://core.trac.wordpress.org/browser/tags/3.3/wp-includes/class-wp-admin-bar.php#L322

Not sure which one is better (iphone looks more descriptive), but it would probably make sense to be consistent.

[markoheijnen]

To be honest the whole iPhone check should be replaced to a mobile check. Since a lot of the things also will apply for other OS's like Android. It seems that this code should apply to a certain device width.

[azaozz]

Replying to markoheijnen:

To be honest the whole iPhone check should be replaced to a mobile check.

Related: ​http://core.trac.wordpress.org/attachment/ticket/20014/20014-vars.php.patch

It seems that this code should apply to a certain device width.

Yes, most of it but perhaps not making the "action links" that are displayed on hover visible by default.

[SergeyBiryukov]

Refreshed the patch.

[SergeyBiryukov]

Refreshed in view of [20417].

[azaozz]

In [20430]:

Move mobile devices CSS from wp-login.php to wp-admin.css, props SergeyBiryukov, fixes #19673