Make WordPress Core

Opened 18 years ago

Closed 14 years ago

#8952 closed enhancement (fixed)

permalink first time customization form trap

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

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 16 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
18 years ago

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

#2 @mrmist
18 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
18 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 2.8 → Future Release

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

#6 @Denis-de-Bernardy
17 years ago

  • Component Permalinks → UI
  • Owner ryan removed

#7 @Denis-de-Bernardy
17 years ago

  • Type defect (bug) → enhancement

@solarissmoke
16 years ago

#8 @solarissmoke
16 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 Future Release → 3.5

#11 @nacin
14 years ago

  • Keywords ui-feedback commit added
  • Owner set to lessbloat
  • Status new → 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 → fixed
  • Status assigned → 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.