WordPress / wordcamp.org

WordCamps are casual, locally-organized conferences covering everything related to WordPress.
https://wordcamp.org
130 stars 74 forks source link

Favorite Session Enhancement #1309

Open renintw opened 6 months ago

renintw commented 6 months ago

Description

It's a great feature to add favorite sessions and send them as an email because often there is a period between when the schedule is released and the actual event day. On the event day, I often find that many people forget which sessions they were interested in at that time, and they have to browse through them again. However, I don't think many people are aware of this feature. On some sites, the star is even used as a decoration, or the 'send to email' widget is completely missing, or the UI is broken.

Here are a few ideas:

  1. Improve the design of the stars: They could be hollow; currently, they are gray, but on some sites, they turn blue, which often makes them seem decorative.
  2. Improve the UI of the widget:
    • Maybe displaying the user's selected favorite sessions can be one of the functions?
    • Show the actual reason for email failures (currently, it only shows 'send fail'), such as the event has ended, or you have not selected any sessions yet.
    • Make the widget's appearance more attractive and noticeable, maybe consider making the overall design to be more modern.
    • [Bug] Email with trailing space should be properly handled.
    • [Bug] On some sites (WCEU for example), it doesn't seem to work correctly.

Screenshots / Screencasts

Add favorite sessions

favorite

Failure message isn't clear enough

Screenshot 2024-05-02 at 15 48 00

Stakeholders

WordCamp Attendees.

How does this new feature contribute to the objectives

Is this a feature that people need? I believe so, but perhaps not a high-priority one. Can it truly achieve your desired outcomes? I think so, it makes it more convenient for attendees, I think this is also one of the reasons why the feature to add favorite sessions was implemented initially.

Acceptance Criteria

Pending discussion.

Priority

Pending discussion.

ciudadanoB commented 6 months ago

Hi @renintw!

Regarding the bug in WCEU site (and others) where it seems that this is not working correctly, it's just a cosmetic thing. Right now you can select some sessions and although the star goes back to grey, if you print your favorite sessions, they all appear.

So it's a CSS thing. If in the file favorite-sessions.css, line 93 you add this second element, they'll appear in yellow correctly:

td.wcb-favourite-session a.fav-session-button,
.wcb-favourite-session .wcb-session-favourite-icon a.fav-session-button {
    color: #ffb900;
}

Now, with this it works, but when you try to deselect one, it keeps the yellow color as it's in the :focus state. I've added some lines to make it work a little better when hovering and focusing that button:

td.wcb-favourite-session a.fav-session-button,
.wcb-favourite-session .wcb-session-favourite-icon a.fav-session-button,
.wordcamp-schedule div.wcb-session-favourite-icon a.fav-session-button:hover,
.wordcamp-schedule div.wcb-session-favourite-icon a.fav-session-button:focus:hover,
.wordcamp-schedule .wcb-favourite-session div.wcb-session-favourite-icon a.fav-session-button:focus {
    color: #ffb900;
}

.wordcamp-schedule .wcb-favourite-session div.wcb-session-favourite-icon a.fav-session-button:hover,
.wordcamp-schedule div.wcb-session-favourite-icon a.fav-session-button:focus {
    color: #aaa;
}

I think this is enough for now to solve this issue and before you make a revamp of the feature or make it look nicer (which I totally agree it's something we should do). Please test it (we're probably adding this code to our custom CSS in WCEU site, but it happens also in other sites like Porto and Vigo) and feel free to modify the CSS and deploy it. Thanks!