Segnalazione #72
App polygen
0%
Description
Porting del vecchio plugin polygen
Associated revisions
Use reverse_lazy. refs: #72
Catch all things in the url, and make some validation in the view. refs: #72
Draft to allow to navigate in the path. refs: #72
Show items in the template. refs: #72
Cleanup. refs: #72
Get separated variables. refs: #72
Implement the view. refs: #72
Template for polygen:list view. refs: #72
Add a couple of auxiliary templatetags to join strings and paths in the templates. refs: #72
To avoid ugly urls with two concatenated '/' characters, get all path in one variable. refs: #72
Use utility_tags and concatenate the paths with os_path_join. refs: #72
Use only buttons to go back. refs: #72
Add split_path() method to the view. refs: #72
Use the method. refs: #72
Fix almost everything in 'polygen:list' view. refs: #72:
- Move path splitting in the dispatch() method
* Redirect to polygen root with error message in case of path not
found in tree (just a 404 was ugly from the UI/UX point of view,
even if it's better from the HTTP point of view. Just in case,
the call is in the code, commented out) * Ugly 'hack' that creates self.root_tree also in List.dispatch()
(see the FIXME comment in the code/diff) and allows the behaviour
explained at the previous point
Cleanup. refs: #72
Merge branch 'master' into t72. refs: #72
Sort these lists. refs: #72
refs #72
small ui improvements
Refactoed without code duplication in dispatch. Refs #72
History
Updated by Mark Caglienzi over 7 years ago
- Status changed from Nuovo to In elaborazione
- Assignee set to Mark Caglienzi
Updated by Mark Caglienzi over 7 years ago
- Status changed from In elaborazione to Commenti
- Assignee changed from Mark Caglienzi to Christopher R. Gabriel
- Voce visibile nella sidebar, con la sua icona
- La home dell'app è l'elenco delle lingue, a cui segue, dopo clic, l'elenco delle grammatiche di quella lingua
- Nella pagina dell'elenco delle grammatiche c'è un bottone back per tornare indietro
- Cliccando su una grammatica si genera una entry per quella grammatica, e nella pagina si aggiunge anche il bottone per generarne un'altra
- Se si scrive un'url inesistente, messaggio di errore e redirect alla home dell'app (restituire semplicemente 404 è più corretto dal punto di vista dello standard HTTP e REST, ma dal punto di vista della UI la pagina è davvero brutta)
- Magari si può fixare dopo aver rifatto la pagina 404 standard. Altro inconveniente è che il messaggio di errore in questo caso viene mostrato dopo che si ricarica una pagina "buona" e non nella 404 stessa. Questo potrebbe confondere l'utente.
Mergiato in master
(dopo merge di master
in t72
e risoluzione di un conflitto facile in octonet/urls.py
) e push.
Updated by Christopher R. Gabriel over 7 years ago
- Assignee changed from Christopher R. Gabriel to Enrico Zini
Per me ok, ho aggiungo qualche fix per la UI e basta.
Riassegno ad Enrico per la review della implementazione, per parlarne poi con Mark.
Updated by Enrico Zini over 7 years ago
- Status changed from Commenti to Chiuso
Direi che ci sia tutto. Ho pushato in master un refactoring della dispatch per evitare duplicazione di codice con OctonetMixin
Draft Polygen app's skeleton. refs: #72