raamdev / wp-redirects

Redirects (a new Post Type) for WordPress®.
1 stars 2 forks source link

Improve Redirect Configuration meta box field arrangement #13

Closed raamdev closed 10 years ago

raamdev commented 10 years ago

The Redirect Configuration meta box currently has the (optional) Redirection Status field at the top. That doesn't make since from a UX-perspective. Since that field is optional, it will most likely rarely get used. The second field (Redirection URL) should be the first field, and the Redirection Status field should be the last field.

2014-09-11_23-06-05


In fact, perhaps each of those fields (Redirection URL, Additional Redirections From, and Redirection Status) should be their own Meta Boxes. That way, a site owner can show/hide the ones they use the most. Separating them into separate Meta Boxes would, in my opinion be fine, since the entire page itself is dedicated to the Redirect Configuration. Also, that existing Redirect Configuration meta box feels very crowded.

@jaswsinc Thoughts?

jaswrks commented 10 years ago

I agree on all points!

Another thing that bugs me about the current layout, is that the tab order seems really out of whack (for whatever reason). I type the title of the Redirect into the field provided, and then I tab down into something other than what I expect. Not sure if we can do anything about this, but if so that would help too I think. It should be quicker and easier to create a redirection.

raamdev commented 10 years ago

The Redirect configuration options have been separated into separate Meta Boxes. I also added lots of inline documentation to help novice site owners.

2014-09-13_20-52-59 2014-09-13_20-53-13

raamdev commented 10 years ago

Another thing that bugs me about the current layout, is that the tab order seems really out of whack (for whatever reason). I type the title of the Redirect into the field provided, and then I tab down into something other than what I expect. Not sure if we can do anything about this, but if so that would help too I think. It should be quicker and easier to create a redirection.

I looked into this and unfortunately there's no easy fix. See https://www.google.com/search?q=wordpress+metabox+tabindex

raamdev commented 10 years ago

@jaswsinc Any feedback here? If it looks good to you, I'll go ahead and merge this. :)

jaswrks commented 10 years ago

Wow, awesome!!

I looked into this and unfortunately there's no easy fix. See https://www.google.com/search?q=wordpress+metabox+tabindex

Gotcha :-)

@jaswsinc Any feedback here? If it looks good to you, I'll go ahead and merge this. :)

I'm sure it's fine. I am going to pull this branch and run tests in just a moment. I will report back shortly.

jaswrks commented 10 years ago

Tested... Yep, looks like pure awesomeness to me! :-) :thumbsup:

raamdev commented 10 years ago

Great! Thanks @jaswsinc :)

raamdev commented 9 years ago

Next release changelog: