| ifried | |
| Aug 9 2022, 4:01 PM |
| F35498027: Screen Shot 2022-08-30 at 12.29.54 PM.png | |
| Aug 30 2022, 4:30 PM |
| F35456504: Screen Shot 2022-08-15 at 4.42.45 PM.png | |
| Aug 15 2022, 10:07 PM |
| F35456500: Screen Shot 2022-08-15 at 4.39.16 PM.png | |
| Aug 15 2022, 10:07 PM |
| F35456502: Screen Shot 2022-08-15 at 4.40.06 PM.png | |
| Aug 15 2022, 10:07 PM |
| F35456441: Screen Shot 2022-08-15 at 2.58.03 PM.png | |
| Aug 15 2022, 7:59 PM |
| F35456439: Screen Shot 2022-08-15 at 2.55.17 PM.png | |
| Aug 15 2022, 7:59 PM |
As an event participant, I want the 'back' link hidden on the EventDetails page, since it is currently only designed for organizers and it is a confusing user experience for participants.
Acceptance Criteria:
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Hide back link on EventDetails for non-organizers | mediawiki/extensions/CampaignEvents | master | +11 -12 |
@ifried I realized that EventDetails already has a button for going to the event page, are we OK with that? If so, I would need the label of the back button: right now it says "back to your events" but that is not suitable if we make it point to the event page.
Hmmm, good point, @Daimona! Perhaps it is better to then just hide the 'back' button for participants until we have a MyEvents page for them too?
Also pinging @gonyeahialam to see what he suggests.
The reason why the current label contains the destination is that there's no one "previous page". Users could get to EventDetails from MyEvents, from the EventPage, by following a link, by directly typing the URL etc. So IIRC we thought it would be confusing to have a back button that doesn't tell you what it links to.
Thanks for the suggestion, @gonyeahialam! Unfortunately, as mentioned by @Daimona, we do not know the last page visited by the user (it could be EventDetails OR MyEvents). For this reason, we cannot choose this option. Rather, we need to determine a specific page that the user is brought back to. Since we already have a link to go back to the event page via EventDetails (and the user can also go back to the event page via the 'back' link in many browsers), it may seem redundant and unnecessary for the arrow to also point back to the event page. Given this situation, what behavior do you recommend?
For the organizer, I think it still makes sense to go back to 'MyEvents.' For the participant, there is minimal value in them currently going back to 'MyEvents' or the event page. So we may want to consider removing the 'Back your events' link for participants, and only displaying the link in the future when we have a page where the participant can see all of the events they have participated in (or a page that has both events they have organized and participated in). What do you two think?
I agree with keeping the link to MyEvents for organizers and hiding the button for participants.
Change 822642 had a related patch set uploaded (by Daimona Eaytoy; author: Daimona Eaytoy):
[mediawiki/extensions/CampaignEvents@master] Hide back link on EventDetails for non-organizers
@Daimona Special:EventDetails/{id} is still displaying the Back to your events button for participants (and also for when logged out, which it should possibly not display for either?)
participants:
logged out:
@vaughnwalters The patch was not merged yet. I think the reason why this task is in QA is for you to checkout the patch locally, test it, and approve it on gerrit as per our latest discussions. Pinging @cmelo to confirm, though.
Ahh yes, you are right! It is functioning correctly on the patch branch.
organizer:
participants:
logged out:
@vaughnwalters Thanks for confirming! @cmelo What should the process be now? Is Vaughn supposed to +2, or should an engineer do that?
Change 822642 merged by jenkins-bot:
[mediawiki/extensions/CampaignEvents@master] Hide back link on EventDetails for non-organizers
@Daimona I can't add a +2 (I only have +1 permissions) so after I give a +1, then claudio will give a +2
The back button in EventDetails has been removed for participants (see screenshot example below). I'm marking this as Done.
