Closed
Bug 970928
Opened 11 years ago
Closed 11 years ago
Update layout / UX for events detail
Categories
(Webmaker Graveyard :: Events, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: cassie, Assigned: gvn)
References
Details
(Whiteboard: events2, frontend, mvp)
Attachments
(6 files)
288.67 KB,
image/png
|
kate
:
review+
|
Details |
476.15 KB,
image/jpeg
|
kate
:
review+
|
Details |
24 bytes,
text/plain
|
cassie
:
review+
gvn
:
feedback+
|
Details |
95 bytes,
text/plain
|
cassie
:
review-
|
Details |
52 bytes,
text/x-github-pull-request
|
kate
:
review-
|
Details | Review |
52 bytes,
text/x-github-pull-request
|
kate
:
review+
|
Details | Review |
Gavin, where are you at with updating the UX for the details page?
The design was resolved in bug 963611.
Reporter | ||
Comment 1•11 years ago
|
||
Kate and I discussed this and think we might want to consider how this page looks without an event image, as there is currently no functionality for users to add an image. Perhaps with a big block of color users can choose that has the event title (need to be careful not to compete too much with /Explore design). Also, what might this look like with the map in-page instead of as an external link.
Assignee | ||
Updated•11 years ago
|
Whiteboard: events2, frontend → events2, frontend, mvp
Assignee | ||
Updated•11 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•11 years ago
|
||
Initial designs: https://bugzilla.mozilla.org/show_bug.cgi?id=963611
Assignee | ||
Comment 3•11 years ago
|
||
Initial designs: https://bug963611.bugzilla.mozilla.org/attachment.cgi?id=8365126
Reporter | ||
Comment 4•11 years ago
|
||
Here is a more refined (UI level) version of the wireframes. Hopefully this helps. (NB: am using what is currently on the site for both header and footer nav)
Attachment #8384823 -
Flags: review?(kate)
Reporter | ||
Comment 5•11 years ago
|
||
And here's the desktop view.
Attachment #8384824 -
Flags: review?(kate)
Assignee | ||
Comment 6•11 years ago
|
||
Initial stab w. available data for MVP:
https://redpen.io/j8ymzt
https://redpen.io/p1ksbe
https://redpen.io/ymn61q
Flags: needinfo?(cassie)
Assignee | ||
Comment 7•11 years ago
|
||
Cassie, do you want to put your comps in redpen too so I can give you feedback?
Comment 8•11 years ago
|
||
Comment on attachment 8384824 [details]
Events-Detail.jpg
Looks good to me as MVP, I added comments in redpen https://redpen.io/h5y3zj
I'm going to mark as a plus, butmaybe after one more iteration let's get this implemented
Updated•11 years ago
|
Attachment #8384823 -
Flags: review?(kate) → review+
Comment 9•11 years ago
|
||
In the mobile view, the blue edit button looks a bit awkward visually with that background colour
Comment 10•11 years ago
|
||
Overall impression: if you are visiting the page directly from twitter or something, it is difficult to tell immediately what the subject matter is (i.e. that it's an event made by an individual contributor)
Updated•11 years ago
|
Flags: needinfo?(cassie)
Updated•11 years ago
|
Attachment #8384824 -
Flags: review?(kate) → review+
Comment 11•11 years ago
|
||
I incorporated some of the feedback into this iteration, some notes:
- the avatar will have to be captured on creation if we are going to include it. Let's do it, it won't be that hard.
- I removed stuff like tags for web literacy but I thought it looked good in the original design. We should file those improvements + the mockup to make sure they get implemented in a future sprint
Attachment #8385436 -
Flags: review?(cassie)
Attachment #8385436 -
Flags: feedback?(gavin)
Comment 12•11 years ago
|
||
Also, i didn't include the mobile version because i think the changes are pretty obvious to the first iteration
Reporter | ||
Comment 13•11 years ago
|
||
Comment on attachment 8385436 [details]
https://redpen.io/naxj6q
Looks good, left a few things to consider in Redpen.
Attachment #8385436 -
Flags: review?(cassie) → review+
Reporter | ||
Comment 14•11 years ago
|
||
I filed an additional bug to capture the competencies – bug 979418 – which I added to the events2 fast follow sprint. I think we need to work hard to build the web lit UX into events as quickly as we can so we can start testing the hypothesis and connecting various pieces of the site.
Assignee | ||
Comment 15•11 years ago
|
||
I'm going to file the Related Events block as a new ticket. It's a reasonably involved feature.
https://bugzilla.mozilla.org/show_bug.cgi?id=979521
Assignee | ||
Comment 16•11 years ago
|
||
Regarding the avatar, shouldn't we be using the user's avatar from their Webmaker account? Or are you saying you should be able to also set your Webmaker avatar from the event creation form?
If we do this, let's make a new ticket.
Flags: needinfo?(kate)
Comment 17•11 years ago
|
||
We should automatically read the user's avatar when a new event gets created. I agree, this should be a different ticket (filed under R2, https://bugzilla.mozilla.org/show_bug.cgi?id=979534)
Flags: needinfo?(kate)
Assignee | ||
Updated•11 years ago
|
Attachment #8385436 -
Flags: feedback?(gavin) → feedback+
Assignee | ||
Comment 18•11 years ago
|
||
Attachment #8386997 -
Flags: review?(cassie)
Assignee | ||
Comment 19•11 years ago
|
||
Comment on attachment 8386997 [details]
Redpens
Here's what I've built for MVP:
960: https://redpen.io/45p2h8
320: https://redpen.io/4vqkne
Reporter | ||
Comment 20•11 years ago
|
||
Comment on attachment 8386997 [details]
Redpens
Left some comments on redpen. Looking good, close!
Attachment #8386997 -
Flags: review?(cassie) → review-
Assignee | ||
Comment 21•11 years ago
|
||
Attachment #8387178 -
Flags: review?(kate)
Comment 22•11 years ago
|
||
Comment on attachment 8387178 [details] [review]
https://github.com/mozilla/webmaker-events-2/pull/26
Code looks good, a few comments in the PR
I think having the flexible width looks off/decreases readability/makes it harder to find things on the right side of the screen
https://www.dropbox.com/s/mulhxsvbdmv2s23/Screenshot%202014-03-06%2018.59.01.png
Attachment #8387178 -
Flags: review?(kate) → review-
Updated•11 years ago
|
Attachment #8387256 -
Flags: review?(kate) → review+
Assignee | ||
Updated•11 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•