Redesign: display online meeting#11075
Conversation
There was a problem hiding this comment.
Hi @Crashillo, thanks for the job 👍 I leave my review as follows:
- The url of the videoconference showed, must be a link. Take in account that the url could change depending on the settings.
a) If "Embed in meeting page" is marked --> https://meet.jit.si/DecidimTry
b) If "Open in a new tab" --> https://meet.jit.si/DecidimTry
c) If "Open in live event page" --> https://decidim-redesign.populate.tools/processes/Decidim4Dummies/f/168/meetings/1815/live_event
In case of c) scenario, maybe it's better to replace the url for a text like "Join meeting"
I also noticed a wired behaviour with the grid. See the following examples:
a) Link with some dashes:

b) Long link without any dash in the url:

As we are running out of time, maybe we can leave it as it is now and in further iterations rethink this. I don't see this as critical, what do you think @carolromero ?
- There's an blank space between the Author and the description text. I've seen more of these blank spaces, so maybe we can open another issue to tackle this?

Online meeting embed
- The text "This meeting is happening right now" should be bigger, maybe a Heading2
- The rest of the text should not be bold. Following the same styles as the description meeting text.
- The CTA should be "Join Meeting" instead of "Register"
More or less, this is how we would like it to be:

Mobile
- It's kind of messy this section (see the screenshot). I don't know how but we need to enhance this. I remember that at some point in the Figma, the Register CTA was fixed in the bottom of the mobile phone screen, as it was the support CTA button in proposals. Are we going to do this or what could be a solution?

This is being done in #10883. Just for the record, the space you're pointing out is the common separator for each section. |
@NilHomedes We discarded that path a time ago, so I don't think its feasible to think about to retake it now. |
|
@NilHomedes can we move forward this PR to maintainers? |
NilHomedes
left a comment
There was a problem hiding this comment.
Okey, we can move forward to not block you anymore with that PR.
But this is not working properly:
c) If "Open in live event page" --> decidim-redesign.populate.tools/processes/Decidim4Dummies/f/168/meetings/1815/live_event
In case of c) scenario, maybe it's better to replace the url for a text like "Join meeting"
|
OK, thanks, feel free to open an issue so this can be tackle separately. |
…-process * feature/redesign: Redesign: display online meeting (#11075) Redesign: home menu content block (#11048) Redesign: home (#10920) Remove widgets and embeds (#11096) Redesign / Complete card G cell specs (#11094) Redesign: processes assemblies metadata content blocks (#10637) Redesign: vertical margin inside the callouts (#11071) Redesign / Update package-lock (#11091) Use the component name in the sidebar title (#11088) Redesign: layout center (#11068) Redesign: display only public users followings in following profile tab (#11051) Redesign: fix cards (#11072)
🎩 What? Why?
Show online status as it was an address field. Includes as well a mobile version of the calendar-address-map widget and the refactorization of the other uses of the static map.
📌 Related Issues
📷 Screenshots