decidim-archive / design

⚠️ [DEPRECATED] Design of the Decidim framework
1 stars 3 forks source link

CTA Participatory Process Home #161

Closed Xfolchf closed 6 years ago

javierarce commented 6 years ago

Here's our proposal for the CTA:

cta

The button will link to the current phase when there's an action users can do (vote or support, read, RSVP, etc.). For those cases where there isn't a clear action (for example, a return phase that's just started), we suggest to send the users to the phase description in the list of phases.

cc: @decidim/product

xabier commented 6 years ago

thanks @javierarce once we complete identified improvements like this one, we should iterate with participat-tests, right now this looks good to me.

Crashillo commented 6 years ago

PR decidim/decidim#2440

mrcasals commented 6 years ago

As @josepjaume said in the PR:

Hi! Not sure how can we implement this, as a process can have many components and any component several call to actions. Have you thought about an admin interface for this?

I don't think we can infer the action from the code directly, as some features might not care about steps. I agree we should have an admin interface for this, even if it's a dropdown for each step somewhere, but this needs to be set by the user and not inferred by the code IMHO.

mrcasals commented 6 years ago

In a discussion in the related PR, @xabier says this needs to work for assemblies too:

This applies to processes and assemblies. Both host complicated mechanisms (process more than current use of organs) so it is very convenient to orient user actions though CTA button.

https://github.com/decidim/decidim/pull/2440#issuecomment-364161605

Processes have steps, so it's quite simple (I'd say) to modify the step admin UI to add the CTA button text and URL fields there, but assemblies do not have steps, so we'll need the admin UI to be modified to add this. Any idea on how to do this?

Also, I would like to point out that even though this issue is called "CTA Participatory Process Home", it turns out it is also needed for assemblies. I think this should have been specified earlier.

/cc @decidim/product

mrcasals commented 6 years ago

PR has been merged, should we close this?

xabier commented 6 years ago

:+1: