Buttons of forms come in two orders; use cancel first, main action last everywhere
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 20/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript, php
Línea de trabajo
Read text.php, festival.php, and mailinglist.php to locate the edit text, festival settings, and mailing list edit forms. Compare their button order with the stated convention, then place cancel before the main action in each remaining form. The issue links open PR #648, so check its changes to see whether this work is already underway.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
The buttons of a form come in two orders, depending on the screen.
| Main action first, then cancel | Cancel first, then the main action |
|---|---|
Edit text page (text.php) |
Translate text page (text.php) |
Festival settings (festival.php) |
All settings screens (settings.php, 9 places) |
Edit mailing list (mailinglist.php) |
Mailing list addresses and mailing (mailinglist.php) |
Login popup (users.php) |
Edit article, moderate comment (peer_reviewed_article.php) |
Recover account popup (users.php) |
New page popup (pages.php) |
So a user who clicks the button in the usual place sometimes cancels instead of saving.
The recover account popup also looks different from the login popup: a table with unstyled browser buttons instead of the login-wrapper with class="button".
Choice
One order everywhere: cancel (or back) first, the main action last. A destructive action, such as delete, goes before cancel.
- This is the order of the Apple Human Interface Guidelines and of Material Design for dialogs and forms. The design of the Markdown editor (#520) refers to the Apple HIG and its dialogs already use this order.
- Most screens of Hypha already follow it (settings, translate, article, new page).
- Themes put the buttons in document order (the default theme does not float them), so the order in the code is the order on screen.
Fix
text.php(edit),festival.php(settings) andmailinglist.php(edit): cancel before save.- The login form: cancel before log in. Done in PR #646, which reworks that form.
- Recover account: the layout of the login form, with cancel first. Done in PR #644, which reworks that popup.
- Lenguaje dominante
- JavaScript
- Estrellas
- 5
- Forks
- 9
- Merge medio
- 18 h 49 min
- PR fusionados (30 d)
- 13
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de PlanBCode/hypha
-
Component: User interface Impact: Low Level: Easy Pagetype: Peer reviewed article Type: Bug Value: Visitors
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
Impact: Medium Level: Easy Type: Bug Value: Users
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
Impact: Low Level: Easy Pagetype: Festival Type: Bug Value: Coders
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
Impact: Medium Level: Easy Pagetype: Mailinglist Type: Bug Value: Users
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
Los mantenedores suelen responder en 1 día
-
Impact: Low Level: Easy Type: Bug Value: Coders
Dificultad 1/5 1-3 horas Aptitud para principiantes 85/100
PlanBCode/hypha#586 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de PlanBCode/hypha
Issues similares
-
bug namespace v1 blocker
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
oakmac/standard-clojure-style-js#269 ·
Los mantenedores suelen responder en 1 día
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
CakeRepository/agentshangout#12 ·
Los mantenedores suelen responder en 1 día
-
JS/TS: JSX props compile errors should link to a doc page explaining the static-list requirementAbiertoarea: js-runtime enhancement exists-upstream interop priority: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
shayanhabibi/fickl#307 ·
Los mantenedores suelen responder en 1 día
-
editor good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
webarkit/webarkit-studio#16 ·
-
agent/dedupe stage/045-needs-decision
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
projectbluefin/documentation#1637 ·
Los mantenedores suelen responder en 1 día