Více orgů k jednomu PDF v korekturovátku #52

Merged
zelvuska merged 4 commits from vice_orgu_v_korekturovatku into master 1 month ago
ledoian commented 6 months ago
Owner

Tož, featura se implementovala. Lokálně vypadá elementárně funkčně. Má ale několik nedostatků:

  • Nic orgům nebrání teď vybírat orgy úplně náhodně. To je ale malý problém, do teď to taky nebránilo.
  • Implementace má implicitní vazebnou tabulku, s explicitní neumím udělat horizontální widget na výběr orgů, jen pomocí inlinů. (Defaultní <select multiple> nemá ani vyhledávání :-/) autocomplete nemůže fungovat (spadne na systémovém checku)
  • Inliny by možná byly trochu hezčí řešení, ale jsou až úplně dole a to je neintuitivní imho
  • Výběr orgů zabírá hodně místa, takže se mi přidání PDF nevejde na jeden monitor
  • v přehledu PDF v adminu nejsou teď vidět autoři. Dá se doimplementovat property, která by se zobrazila, ale z dokumentace se zdálo, že by se vykonal DB dotaz pro každou jednu položku, což je trochu nechutné: https://docs.djangoproject.com/en/5.0/ref/contrib/admin/#django.contrib.admin.ModelAdmin.list_display

Možná můžeme schovat věci typu datum a počet stran do nějakého skrytého fieldsetu, tím se problém asi trochu omezí. Případně do toho žďuchat ještě dál/jinak, nevím.

Možná se dá prozkoumat, jak se používají generické vazby jako inline, ale moc tomu nedávám… https://docs.djangoproject.com/en/5.0/ref/contrib/admin/#using-generic-relations-as-an-inline

Tož, featura se implementovala. Lokálně vypadá elementárně funkčně. Má ale několik nedostatků: - Nic orgům nebrání teď vybírat orgy úplně náhodně. To je ale malý problém, do teď to taky nebránilo. - Implementace má implicitní vazebnou tabulku, s explicitní ~~neumím udělat horizontální widget na výběr orgů, jen pomocí inlinů. (Defaultní `<select multiple>` nemá ani vyhledávání :-/)~~ autocomplete nemůže fungovat (spadne na systémovém checku) - ~~Inliny by možná byly trochu hezčí řešení, ale jsou až úplně dole a to je neintuitivní imho~~ - ~~Výběr orgů zabírá hodně místa, takže se mi přidání PDF nevejde na jeden monitor~~ - v přehledu PDF v adminu nejsou teď vidět autoři. Dá se doimplementovat property, která by se zobrazila, ale z dokumentace se zdálo, že by se vykonal DB dotaz pro každou jednu položku, což je trochu nechutné: <https://docs.djangoproject.com/en/5.0/ref/contrib/admin/#django.contrib.admin.ModelAdmin.list_display> Možná můžeme schovat věci typu datum a počet stran do nějakého skrytého fieldsetu, tím se problém asi trochu omezí. Případně do toho žďuchat ještě dál/jinak, nevím. Možná se dá prozkoumat, jak se používají generické vazby jako inline, ale moc tomu nedávám… <https://docs.djangoproject.com/en/5.0/ref/contrib/admin/#using-generic-relations-as-an-inline>
ledoian added 1 commit 6 months ago
Poster
Owner

Ha, ono to umí i Select2Multiple. Půlka problémů tím padá…

Ha, ono to umí i Select2Multiple. Půlka problémů tím padá…
ledoian added 1 commit 6 months ago
Poster
Owner

Jen nevím, jestli autocomplete dodržuje pořadí, tj. jestli nabízí aktivní orgy na začátku. Zrovna tady to asi hodně chceme, TODO otestovat na testwebu. (Teď už to dělat nebudu.)

Jen nevím, jestli autocomplete dodržuje pořadí, tj. jestli nabízí aktivní orgy na začátku. Zrovna tady to asi hodně chceme, TODO otestovat na testwebu. (Teď už to dělat nebudu.)
ledoian added 1 commit 5 months ago
zelvuska added 1 commit 1 month ago
Owner

Vypadá funkčně, proti kódu nic nemám :)

Vypadá funkčně, proti kódu nic nemám :)
Owner

Jen nevím, jestli autocomplete dodržuje pořadí, tj. jestli nabízí aktivní orgy na začátku. Zrovna tady to asi hodně chceme, TODO otestovat na testwebu. (Teď už to dělat nebudu.)

Ano dodržuje :)

> Jen nevím, jestli autocomplete dodržuje pořadí, tj. jestli nabízí aktivní orgy na začátku. Zrovna tady to asi hodně chceme, TODO otestovat na testwebu. (Teď už to dělat nebudu.) Ano dodržuje :)
zelvuska merged commit c7910ed72a into master 1 month ago
zelvuska deleted branch vice_orgu_v_korekturovatku 1 month ago
The pull request has been merged as c7910ed72a.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.