mam/mamweb
11
0
Fork 0
Code Issues Pull requests 9 Projects Wiki Activity Actions

Upgrade odevzdavatka #30

Merged
zelvuska merged 37 commits from upgrade_odevzdavatka into master 2023-06-19 20:48:30 +02:00
Conversation 3 Commits 37 Files changed 15 +8 -3
2 changed files with 8 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit dce1de7a99 - Show all commits

7
mamweb/static/css/mamweb.css
View file

@ -1264,16 +1264,17 @@ label[for=id_skola] {
/* Select2 používaný hlavně multiple selectem. Přidání checkboxů a změna barvy. */
/* Podle https://stackoverflow.com/a/48290544 */
.select2-results__option[aria-selected=true]:before {
/* U autocomplete.ModelSelect2Multiple vyžaduje 'data-dropdown-css-class': 'select2multiple' */
.select2multiple .select2-results__option[aria-selected=true]:before {
content: '☑ ';
padding: 0 0 0 8px;
}
.select2-results__option[aria-selected=false]:before {
.select2multiple .select2-results__option[aria-selected=false]:before {
content: '◻ ';
padding: 0 0 0 8px;
}
.select2-results__option--highlighted {
.select2multiple .select2-results__option--highlighted {
background-color: #e84e10 !important;
}
zelvuska marked this conversation as resolved
ledoian commented 2023-06-19 18:58:19 +02:00
Review
Copy link

Přemýšlím, kde všude jsou Select2Multiple, jestli je to nerozbíjí. Ale snad ne.

Přemýšlím, kde všude jsou Select2Multiple, jestli je to nerozbíjí. Ale snad ne.
zelvuska commented 2023-06-19 19:11:25 +02:00
Review
Copy link

Tak tohle je jen tam, kam jsme explicitně přidali 'data-dropdown-css-class': 'select2multiple'. Spíš si teď naopak říkám, že ta barva možná měla být bez toho .select2multiple

Tak tohle je jen tam, kam jsme explicitně přidali `'data-dropdown-css-class': 'select2multiple'`. Spíš si teď naopak říkám, že ta barva možná měla být bez toho `.select2multiple`
ledoian commented 2023-06-19 19:54:49 +02:00
Review
Copy link

Měl jsem na mysli obecně všechny Select2Multiple, nemělo se to týkat jen barvy…

Barva asi může být všude (v Adminu tyhle styly stejně nepoužíváme, ne?)

Měl jsem na mysli obecně všechny Select2Multiple, nemělo se to týkat jen barvy… Barva asi může být všude (v Adminu tyhle styly stejně nepoužíváme, ne?)
zelvuska commented 2023-06-19 19:58:26 +02:00
Review
Copy link

No ale tohle neovlivňuje všechny selecty. Tohle ovlivňuje jen ty, kterým tu třídu předhodíme…

No ale tohle neovlivňuje všechny selecty. Tohle ovlivňuje jen ty, kterým tu třídu předhodíme…
zelvuska commented 2023-06-19 19:58:44 +02:00
Review
Copy link

A barvu bych tedy dal všude…

A barvu bych tedy dal všude…
ledoian commented 2023-06-19 20:07:22 +02:00
Review
Copy link

Třída .select2multiple je naše? Tak bych ji tím pádem pojmenoval nějak aspoň parciálně česky, myslel jsem, že hackujeme nějakou vestavěnou. (Uznávám, nečetl jsem kód moc pozorně…)

Možná .s2m-se-zaskrtavatky?

Třída `.select2multiple` je naše? Tak bych ji tím pádem pojmenoval nějak aspoň parciálně česky, myslel jsem, že hackujeme nějakou vestavěnou. (Uznávám, nečetl jsem kód moc pozorně…) Možná `.s2m-se-zaskrtavatky`?

4
odevzdavatko/forms.py
View file

@ -30,6 +30,7 @@ class PosliReseniForm(forms.Form):
'data-placeholder--id': '-1',
'data-placeholder--text': '---',
'data-close-on-select': 'false',
'data-dropdown-css-class': 'select2multiple',
'data-allow-clear': 'true'
},
),
@ -45,6 +46,7 @@ class PosliReseniForm(forms.Form):
attrs = {'data-placeholder--id': '-1',
'data-placeholder--text' : '---',
'data-close-on-select': 'false',
'data-dropdown-css-class': 'select2multiple',
'data-allow-clear': 'true'})
)
@ -77,6 +79,7 @@ class NahrajReseniForm(forms.ModelForm):
attrs = {'data-placeholder--id': '-1',
'data-placeholder--text' : '---',
'data-close-on-select': 'false',
'data-dropdown-css-class': 'select2multiple',
'data-allow-clear': 'true'},
forward=["nadproblem_id"],
),
@ -86,6 +89,7 @@ class NahrajReseniForm(forms.ModelForm):
attrs = {'data-placeholder--id': '-1',
'data-placeholder--text' : '---',
'data-close-on-select': 'false',
'data-dropdown-css-class': 'select2multiple',
'data-allow-clear': 'true'},
)
}