Opened 14 years ago
Closed 14 years ago
#3178 closed defect (bug) (fixed)
Don't show BP-Wizard permalinks panel if we have pretty permalinks
Reported by: | cnorris23 | Owned by: | |
---|---|---|---|
Milestone: | 1.5 | Priority: | normal |
Severity: | Version: | 1.5 | |
Component: | Core | Keywords: | has-patch |
Cc: |
Description
During a new installation of BP, the permalinks panel is shown regardless of whether or not pretty permalinks are already in use. This can cause confusion if the user already has them set.
Attachments (3)
Change History (13)
#1
@
14 years ago
- Milestone changed from Awaiting Review to 1.3
- Owner set to DJPaul
- Status changed from new to assigned
#5
@
14 years ago
- Keywords has-patch removed
- Resolution fixed deleted
- Status changed from closed to reopened
I decided to revert this commit as it had the unintended consequence of making the "Permalinks" panel disappear after they were set in the install. This looked a bit jarring. A better solution would be to improve the text on the permalinks panel to make it clear that they've already been set. Most of the the upgrader screens' text need some love, so it's probably worth getting them in one go.
#6
@
14 years ago
I remember running into this when I was doing something with the pages panel. Unfortunately, I can't remember how I fixed it. I'll play with it tonight, and if I can't come up with a solution that's not hacky, I'll write up a patch with alternate text.
#7
@
14 years ago
- Keywords has-patch added
Finally got to this. I spent some time trying to figure something out, but there's no easy way that doesn't require a temporary database option or another cookie. It's just not worth all that. Besides, it can't hurt to give admins another chance to get the pretty permalinks of their dreams :)
Nice idea