search

dbrugne / donut

The donut server and browser based client
http://donut.me
0 stars 0 forks source link

Restore and improve group create confirmation popin #1212

Open dbrugne opened 8 years ago

dbrugne commented 8 years ago

@hageged Un popin de confirmation est affiché quand je créer un groupe.

1 - c'est compliqué à gérer et ça rends le code plus ardus pour rien 2 - c'est moche 3 - le contenu me fait pas tomber de ma chaise

Peut on :

Pour le moment il ne s'affiche plus car j'ai dégagé le bout de code de la logique de routage (views/drawer-group-create.js:91 et views/nav-groups.js:115).

hageged commented 8 years ago

@dbrugne tu pourrais me copier coller ici le contenu de la pop-in ? Ca me permettrait de produire un deuxième jet au lieu de repartir de zéro.

dbrugne commented 8 years ago 
  "popins": {
    "group-create": {
      "title": "Community's creation",
      "content": "Congrats ! This is your homepage where your members can see all the members, and all the donuts of your community.<br/><br/> We created your first donut: <strong>#__groupname__/welcome</strong>. It is public so anyone (even non-members) can join it. Take a minute to <a href=\"#\" class=\"inverse open-group-profile\" data-group-id=\"__groupid__\" data-dismiss=\"modal\">customize your profile</a> and configure who can <a href=\"#\" class=\"inverse open-group-access\" data-group-id=\"__groupid__\" data-dismiss=\"modal\">access your community</a>"
    }
  },
dbrugne commented 8 years ago 
  "popins": {
    "group-create": {
      "title": "Création de communauté",
      "content": "Ca y est, vous avez votre communauté ! Voici votre page d'accueil, vos membres peuvent y voir tous les autres membres, et tous les donuts pour les rejoindre plus facilement.<br/><br/> Nous avons créé votre premier donut: <strong>#__groupname__/welcome</strong>. Il est public donc accessible y compris aux non-membres. Pensez à <a href=\"#\" class=\"inverse open-group-profile\" data-group-id=\"__groupid__\" data-dismiss=\"modal\">personnaliser votre profil</a> et ceux qui <a href=\"#\" class=\"inverse open-group-access\" data-group-id=\"__groupid__\" data-dismiss=\"modal\">accèdent à votre communauté</a>"
    }
  },
dbrugne commented 8 years ago

Ci-dessus @hageged

hageged commented 8 years ago

@dbrugne

Je suis d'avis de supprimer la pop-in et de ne reprendre aucun texte.

Le seul élément qui manque un peu c'est le set up d'Access. Je propose de le compenser en ajoutant un bouton "Access rules" / "Modalités d'accès" en remplacement du bouton Manage members.

En effet ce bouton Manage Members n'a de sens que dans un deuxième temps. C'est suffisant de l'avoir dans le menu de roue crantée.

image