Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[JS] react-select required atribut #39

Open
rodlukas opened this issue Aug 7, 2018 · 0 comments
Open

[JS] react-select required atribut #39

rodlukas opened this issue Aug 7, 2018 · 0 comments
Assignees
@rodlukas
Labels
⏰ waiting

Comments

@rodlukas
Copy link
Owner

rodlukas commented Aug 7, 2018 •
edited

React-select v2 zatím nepodporuje atribut required, tam kde je vyžadován je v kódu ponechán, ale nijak se neprojeví (uživatel není upozorněn) - čeká se na implementaciv react-select, pak se případně doupraví (možná se přejmenuje na isRequired místo required)
@rodlukas rodlukas added 🐜 bug něco nefunguje ⏰ waiting labels Aug 7, 2018
rodlukas added a commit that referenced this issue Aug 7, 2018
@rodlukas
přechod na react-select v2, rozšíření jeho využití na další prvky, zj…
b4a04f5 
…ednodušení kódů prvků formulářů a opravy `label for=""`, v konstantách obsaženy texty

- kompletní přechod na react-select v2, provedení potřebných změn, nový issue #39
- react-select boxy se používají na dalších místech (výběry kurzu)
- v konstantách je proměnná `TEXTS` obsahující texty používané na více místech aplikace (zatím jen "Nic nenalezeno" pro react-select)
- ve většině formulářových prvků odstraněn atribut `name` pro svou nadbytečnost a duplicitu
- úpravy atributů formulářů `id` a `for` (někde byly špatně, někde chyběly apod.) -> formuláře teď plně všude podporují kliknutí na popisek
@rodlukas rodlukas self-assigned this Aug 7, 2018
@rodlukas rodlukas removed the 🐜 bug něco nefunguje label Nov 18, 2019
@rodlukas rodlukas added this to todo pool in DP Feb 7, 2020
@rodlukas rodlukas moved this from todo pool to možná rozšíření v budoucnu in DP Apr 12, 2020
@rodlukas rodlukas removed this from možná rozšíření v budoucnu in DP Apr 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rodlukas rodlukas

Labels
⏰ waiting
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@rodlukas