On our site we allow our "Support Roles" to have unlimited invites.. for some of them they have 100's or so.

I would like to recommend that rather then just listing all invites we do 1 of a couple of things:

1) Use a pager so it only shows so many invites at a time and the status
2) Add a tab for each status of the invite. So if a user wants to see only pending invites or only accepted invites they can easily
3) Add a filter to filter out only those invite status they wish to see

I think this would help in the UI and would at least clean things up. Optionally I would also like to have an admin setting that if a user accepted a link to the user page would show up .

CommentFileSizeAuthor
#4 invite_ui_0.patch9.61 KBsmk-ka
#1 invite_ui.patch10.18 KBsmk-ka

Comments

smk-ka’s picture

Assigned: Unassigned » smk-ka
Status: Active » Needs review
StatusFileSize
new10.18 KB

Oh yes, it is definitely time for a clean-up. I will probably get nuked for removing the customizable help text, but with the three separate tabs (and some added logic) I felt it was time to do so. It's also bad for localization.

This patch implements no.'s 1 & 2 from the list. No. 3 is kinda pointless, as everything is on a separate tab now.

Please do some testing and tell me what you think of it.
--
Stefan Kudwien
unleashed mind

khoogheem’s picture

Works like a charm... Yes 3 is pointless with the changes made in the patch. Everything seems to work well.

The only clean up I would do at this point is..
If you have a user that has sent out NO invites either take away the tab "Your Invites" or change the wording, where it says you don't have any accepted invitations.. To you have not sent out any invites

smk-ka’s picture

Removing the Your invites tab is probably not an option, since the Quick invite block also links to that item (and it would be major fuzz to get it displayed one time but not the other).

Which leaves us with the second option: the former text displayed already was quite similar "You have not sent any invitations yet", but since there actually can be some invites on any of the other tabs, I felt the wording was too generic (or simply untrue). Therefore I wanted to reflect that only the current tab contains no data. What about a short "No invitations available"?

smk-ka’s picture

StatusFileSize
new9.61 KB

New patch:
- Fixed the Cancel link on the withdraw confirmation page. It now properly returns you to the originating status tab.
- Changed the text to "No invitations available" for now. This is by no means a final decision, I'd still like to hear your opinion on that issue.

khoogheem’s picture

Works well. There prob is some better wording.. But i think for now it makes sense.

Very nice and clean now with the three extra tabs.

smk-ka’s picture

Status: Needs review » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)