Make WordPress Core

Opened 13 years ago

Last modified 5 years ago

#18287 new enhancement

wp-login.php - Logo URL should be relative (affects MultiSite)

Reported by: ipstenu's profile Ipstenu Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 3.2
Component: Login and Registration Keywords: has-patch
Focuses: multisite Cc:

Description

If you go to your domain.com/wp-login.php page it has a WP logo that links back to domain.com

If you're using multisite and go to subsite.domain.com/wp-login.php, it links back to domain.com

If you're using multisite and you MAP a domain and go to mappeddomain.com/wp-login.php, it links back to domain.com

I can see why this would have been done. It points to network_home_url() which is perfectly sensible for MOST situations. But maybe we should just honor the site on which people are trying to login, so that it's all relative. Given the link on the bottom (back to...) goes to the relative site, this is a logical change. I would say to keep the EMAILS for password resets as is, since the network is in charge of it all, but the front end interface should be consistant.

I did a quick test and this works fine on Single and Multi (as expected). So ... here's my first diff!

Attachments (2)

relative-wp-login.diff (1.0 KB) - added by Ipstenu 13 years ago.
relative URL on wp-login, for MultiSite.
18287.patch (1.5 KB) - added by SergeyBiryukov 13 years ago.
with home_url()

Download all attachments as: .zip

Change History (28)

@Ipstenu
13 years ago

relative URL on wp-login, for MultiSite.

#1 @SergeyBiryukov
13 years ago

  • Keywords has-patch added

Shouldn't it be home_url(), for proper SSL support?

#2 @Ipstenu
13 years ago

The backtoblog link doesn't, so I wasn't sure which ideology would take precedence and went with what's in the more obvious link (I didn't even realize the logo was linked until yesterday). Clearly, it's just as easy a fix either way.

#3 @SergeyBiryukov
13 years ago

  • Keywords ui-feedback added

@SergeyBiryukov
13 years ago

with home_url()

#4 @SergeyBiryukov
13 years ago

  • Keywords ux-feedback added; ui-feedback removed

#5 @bpetty
12 years ago

Makes sense to me that this would link to the home_url() rather than the network home.

Kind of a shame to see such a simple ticket like this sit for 18 months with no attention.

#6 @SergeyBiryukov
12 years ago

  • Milestone changed from Awaiting Review to 3.6

#7 @nacin
12 years ago

  • Type changed from defect (bug) to enhancement

For single-site, it links to http://wordpress.org. For multi-site — given its open registration multi-user history — it links to the network's home.

Based on the single-site behavior, it certainly seems to make sense as-is. There's also a "Back to site" link on the same page, which would become redundant with this change.

I'm fine with a re-think here, but note this isn't just a simple bug-fix.

#8 follow-up: @Ipstenu
12 years ago

Interesting, why does single point to WordPress.org at all?

Seems like they should all point to the same place, and be filterable so a network could change easily it if they wanted.

#9 in reply to: ↑ 8 ; follow-up: @rmccue
12 years ago

Replying to Ipstenu:

Interesting, why does single point to WordPress.org at all?

That's a very good question. It has always seemed like a weird quirk to me personally. +1 for changing across the board.

#10 @mindctrl
12 years ago

  • Cc mindctrl added

#11 in reply to: ↑ 9 @gazouteast
11 years ago

  • Cc gazouteast added

Replying to rmccue:

Replying to Ipstenu:

Interesting, why does single point to WordPress.org at all?

That's a very good question. It has always seemed like a weird quirk to me personally. +1 for changing across the board.

I'm assuming that it links to wp.org because it's the WordPress logo carrying the link.

If the link is to point at network home, then there should be a method to distribute the network home's logo to the wp-login and replace the WordPress logo.

If the link is to point to point to network site home, then there should be a logo from network site that replaces the WordPress logo.

If not, you just know that eventually someone will try using that "accustomed behaviour" of clicking on the WP logo to go to wp.org to create a fake wp.org site (for what ever malevolent or mischevious reason).

An option for site branding / logo could be added to Settings - General.
It's just an instruction to upload a logo of particular dimensions, and if not null, then point to the site/network home. Network admins should have option to decide if network logo & link overrides sub-sites' logos and links (thus switching off the option in sub sites).

#12 @DrewAPicture
11 years ago

  • Keywords dev-feedback added

Seems like there's still some disagreement on whether (A) The link should be consistent for both single and multisite, and (B) Whether we should be linking to WordPress.org or home_url() which feeds back in to (A). Maybe we should punt this and hash it out in 3.7.

#13 @ipstenu
11 years ago

I would say it should like to WordPress for everyone, since that is an understandable expectation based on Single.

Since you can already replace the logo - http://wordpress.org/extend/plugins/login-logo/ - we'll need to make sure that we can replace on the fly (Then Mark's logo would be a perfect way to change the logo from Generic WP to your site, with a link back to Your Site :) )

#14 @nacin
11 years ago

  • Keywords dev-feedback removed
  • Milestone changed from 3.6 to Future Release

There are already filters in place, so yes, replacement is doable.

I agree with not dealing with this now. Maybe a combination of some multisite work and possibly MP6 in 3.7/8 could allow us to revisit.

#15 @jeremyfelt
11 years ago

  • Component changed from Administration to Login and Registration
  • Focuses multisite added

#16 @chriscct7
9 years ago

  • Version changed from 3.2.1 to 3.2

This ticket was mentioned in Slack in #core-multisite by ipstenu. View the logs.


8 years ago

This ticket was mentioned in Slack in #design by karmatosed. View the logs.


7 years ago

#22 @Ipstenu
7 years ago

It does seem like the right time.

This ticket was mentioned in Slack in #design by karmatosed. View the logs.


6 years ago

#24 @JoshuaWold
6 years ago

Howdy! We just discussed this ticket in our #design triage sync. The Tl;DR that I was able to come up with of all the above discussions is:

  1. Do we want the WordPress logo to link to w.org or to the actual homepage of the website that you’re on?
  2. Should things be different on multisite vs single sites?
  3. If I could replace the logo with my sites logo should it then link to my own website instead of w.org?

My opinion, based on reading the description: if it shows a WordPress logo just link to w.org. It’s not perfect, but it’s simple. If you customize it to show your own logo then link to your site’s homepage.

This ticket was mentioned in Slack in #design by karmatosed. View the logs.


5 years ago

#26 @karmatosed
5 years ago

  • Keywords ux-feedback removed

This has ux-feedback, so let's remove the keyword for now.

Note: See TracTickets for help on using tickets.