Make WordPress Core

Opened 17 years ago

Closed 14 years ago

#8952 closed enhancement (fixed)

permalink first time customization form trap

Reported by: jidanni's profile jidanni Owned by: lessbloat's profile lessbloat
Milestone: 3.5 Priority: normal
Severity: normal Version: 2.7
Component: UI Keywords: has-patch commit
Focuses: Cc:

Description

A very big bug for the unfortunate few: The first time one visits the
wp-admin/options-permalink.php page, one sees models like
http://localhost/articles/2009/01/sample-post/ above the custom
structure box, and thus might easily think you want everything,
including "http" to be filled into the custom structure box. This will
give all kinds of problems.

It turns out that one needs to click one of the other buttons to
reveal that all you don't want us to include that much.

You may think "of course" but I'm here to tell you that I fell for it,
and wasted a half an hour figuring out what was wrong.

Attachments (2)

8952.diff (536 bytes) - added by solarissmoke 15 years ago.
8952.after.png (14.2 KB) - added by SergeyBiryukov 14 years ago.

Download all attachments as: .zip

Change History (15)

#1 @Denis-de-Bernardy
17 years ago

it might make sense to add get_option(home) before the text box.

#2 @mrmist
17 years ago

Would probably be less of an issue should #6481 be committed, as then there would be an example in the custom structure box already.

#3 @jidanni
17 years ago

  • Cc jidanni@… added

#4 @Denis-de-Bernardy
17 years ago

agreed. it would also help if Jane wasn't mindlessly punting, too.

#5 @Denis-de-Bernardy
17 years ago

  • Keywords needs-patch added
  • Milestone changed from 2.8 to Future Release

punting to future from lack of patch, and given that #6481 has one

#6 @Denis-de-Bernardy
17 years ago

  • Component changed from Permalinks to UI
  • Owner ryan deleted

#7 @Denis-de-Bernardy
17 years ago

  • Type changed from defect (bug) to enhancement

@solarissmoke
15 years ago

#8 @solarissmoke
15 years ago

  • Keywords has-patch added; needs-patch removed

I think this looks a lot more consistent

#9 @SergeyBiryukov
14 years ago

one sees models like http://localhost/articles/2009/01/sample-post/ above the custom structure box, and thus might easily think you want everything, including "http" to be filled into the custom structure box

This crops up on support forums once in a while.

8952.diff looks like a good enhancement: 8952.after.png.

#10 @nacin
14 years ago

  • Milestone changed from Future Release to 3.5

#11 @nacin
14 years ago

  • Keywords ui-feedback commit added
  • Owner set to lessbloat
  • Status changed from new to assigned

#12 @lessbloat
14 years ago

  • Keywords ui-feedback removed

8952.diff is a nice quick fix. Looks good to me.

#13 @nacin
14 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In [21891]:

Include the home URL before the custom permalink structure input, so it is clear we only care about the path. props solarissmoke, fixes #8952.

Note: See TracTickets for help on using tickets.