Project

General

Profile

Actions

Feature #2726

closed

Reorder Positions in Profile Field

Added by Matt Gold over 10 years ago. Updated about 10 years ago.

Status:
Resolved
Priority name:
Normal
Assignee:
Category name:
Public Portfolio
Target version:
Start date:
2013-08-23
Due date:
% Done:

0%

Estimated time:
6.00 h
Deployment actions:

Description

Boone, is it possible to reorder positions within the Positions field on the new profile pages? If not, can we look into adding that functionality?

Also, we should be able to remove a single position without having to remove the entire widget.

Actions #1

Updated by Matt Gold over 10 years ago

(I say this as someone with, like, 15 positions)

Actions #2

Updated by Boone Gorges over 10 years ago

  • Status changed from Assigned to Reporter Feedback

Making them reorderable is a significant task. Probably 4-6 hours.

You can already delete individual positions. In edit mode, you should see a red x by individual positions.

Actions #3

Updated by Matt Gold over 10 years ago

Making them reorderable is a significant task. Probably 4-6 hours.

I think we need to do it. Otherwise, it's random based on the order in which they were entered.

You can already delete individual positions. In edit mode, you should see a red x by individual positions.

Aha -- thanks. Sorry I missed that.

Actions #4

Updated by Boone Gorges over 10 years ago

  • Tracker changed from Support to Feature
  • Status changed from Reporter Feedback to Assigned
  • Target version set to 1.6
  • Estimated time set to 6.00 h
Actions #6

Updated by Boone Gorges about 10 years ago

  • Category name set to Commons Profile
Actions #7

Updated by Boone Gorges about 10 years ago

  • Category name changed from Commons Profile to Public Portfolio
Actions #8

Updated by Boone Gorges about 10 years ago

  • Status changed from Assigned to Testing Required

The feature is ready to test on cdev. Pretty self-explanatory. Required some slight changes to the UI, but I think they're for the better anyway.

Actions #9

Updated by Matt Gold about 10 years ago

Works beautifully! The only problem for me was that I didn't realize that I'd have to click on the gray bar to move the position (tried instead to grab the white space between College and drop-down). Any thoughts about a way to either 1) make the whole thing draggable, or 2) give a visual cue that one should grab the gray bar to reorder?

Actions #10

Updated by Matt Gold about 10 years ago

bumping this to Boone for a response to my testing feedback

Actions #11

Updated by Matt Gold about 10 years ago

  • Status changed from Testing Required to Assigned
Actions #12

Updated by Boone Gorges about 10 years ago

  • Status changed from Assigned to Reporter Feedback

My opinion is that nothing needs to be done. It's unlikely that anyone would misclick more than one time, and it's not worth adding extra UI. If you disagree, just let me know what you want me to do.

Actions #13

Updated by Matt Gold about 10 years ago

Hi Boone. We can certainly implement as is and iterate for a future version. But to be clear, my questions had less to do with misclicking than with ensuring that people know where to click in order to drag the field. My suggestion was (or was supposed to be) that the left side of the box have some visual cue (a texture, an icon) that would indicate "draggability" on a visual level.

Actions #14

Updated by Boone Gorges about 10 years ago

. But to be clear, my questions had less to do with misclicking than with ensuring that people know where to click in order to drag the field. My suggestion was (or was supposed to be) that the left side of the box have some visual cue (a texture, an icon) that would indicate "draggability" on a visual level.

Yes, I understood that but didn't respond very clearly. My thoughts on visual cues are that none are necessary. The draggable area for positions is very similar to that for the widgets themselves, and at most it takes a few seconds of guessing to try dragging using the draggable area. If you disagree and would like something added, please let me know what it is.

Actions #15

Updated by Matt Gold about 10 years ago

Hi Boone --

what about the solution proposed here - http://ux.stackexchange.com/questions/45457/how-do-you-indicate-an-element-is-moveable-by-keyboard-arrows - ie., changing the cursor to a cross-cursor when it is over the draggable area to indicate that drag and drop is possible? That seems like a reasonable solution to me.

Actions #16

Updated by Chris Stein about 10 years ago

To address Matt's concerns and keep it consistent I suggest:

  1. Style it with the same stripes as on the sections.
  2. change the cursor style to cursor:move;

Overall I think this works well. It's pretty intuitive. I have a one more usability observation:

when I added a new position it scrolled back to the top and I had to scroll back down to work with it. Perhaps an event.preventDefault(); is needed?

Actions #17

Updated by Boone Gorges about 10 years ago

  • Status changed from Reporter Feedback to Testing Required

Chris's two suggestions have been implemented, and I've fixed the scroll bug. Ready to test on cdev.

Actions #18

Updated by Boone Gorges about 10 years ago

  • Assignee changed from Boone Gorges to Chris Stein
Actions #19

Updated by Matt Gold about 10 years ago

  • Status changed from Testing Required to Resolved

Perfect. I think the changed cursor and stripes really help. Marking resolved (Chris, please reopen if you see a need for more work).

Actions

Also available in: Atom PDF