Make WordPress Core

Opened 16 years ago

Closed 12 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 14 years ago.
8952.after.png (14.2 KB) - added by SergeyBiryukov 12 years ago.

Download all attachments as: .zip

Change History (15)

#1 @Denis-de-Bernardy
16 years ago

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

#2 @mrmist
16 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
16 years ago

  • Cc jidanni@… added

#4 @Denis-de-Bernardy
16 years ago

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

#5 @Denis-de-Bernardy
16 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
16 years ago

  • Component changed from Permalinks to UI
  • Owner ryan deleted

#7 @Denis-de-Bernardy
16 years ago

  • Type changed from defect (bug) to enhancement

@solarissmoke
14 years ago

#8 @solarissmoke
14 years ago

  • Keywords has-patch added; needs-patch removed

I think this looks a lot more consistent

#9 @SergeyBiryukov
12 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
12 years ago

  • Milestone changed from Future Release to 3.5

#11 @nacin
12 years ago

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

#12 @lessbloat
12 years ago

  • Keywords ui-feedback removed

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

#13 @nacin
12 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.