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.

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

Dokumentace API Nette\Forms\Container a Nette\Forms\Controls\BaseControl #648

Open
janmottl opened this issue Sep 8, 2017 · 6 comments
Open

Comments

@janmottl
Copy link

janmottl commented Sep 8, 2017

V dokumentaci API Nette\Forms\Container by se hodilo:

  • popsat v metode setDefaults, ze v pripade submitted formu hodnoty nenastavi
  • zanest do dokumentace metodu setValues, ktera neni v dokumentaci API popsana

V dokumentaci API Nette\Forms\Controls\BaseControl by se hodilo:

  • popsat v metode setDefaultValue, ze v pripade submitted formu hodnoty nenastavi
  • popsat v metode settValue, ze funguje i na submited formu
@dg
Copy link
Member

dg commented Sep 8, 2017

Mohl bys to prosím zkusit doplnit a poslat pull request? Na základě připomínek to postupně doladíme a pak vytvoříme anglickou verzi. S tím pomůže Google Translator.

@janmottl
Copy link
Author

janmottl commented Sep 8, 2017

Asi jsem to zapsal na nevhodné místo. Je to návrh do dokumentace API, což bude generované z kódu. Nechci vám dělat nepořádek v kódu a tak posílám návrh textu.

Nette\Forms\Container metoda setDefaults. Za text "Fill-in with default values." doplnit:
Boundary conditions: The form must not be submitted or anchored.
Okrajové podmínky: formulář nesmí být odeslaný (submitted) nebo anchored.

Nette\Forms\Controls\BaseControl metoda setDefaultValue. Za text "Sets control's default value." doplnit:
Boundary conditions: The form must not be submitted or anchored.
Okrajové podmínky: formulář nesmí být odeslaný (submitted) nebo anchored.

@f3l1x
Copy link
Member

f3l1x commented Sep 8, 2017

@janmottl Je to super jak jsi to sepsal, mame tech veci na upravu opravdu hodne, zkus si prosim stahnout dokumentaci k sobe a vlozit tam tvoje upravy. ;-)

Je to snazsi nez se zda.

@janmottl
Copy link
Author

podivam se na to behem tydne jak budu mit cas.

@f3l1x
Copy link
Member

f3l1x commented Oct 26, 2017

ping @janmottl, podarilo se nejak?

@janmottl
Copy link
Author

Omlouvam se, ale nepodarilo. Je to navrh k dokumentaci API. Ta je predpokladam generovana z kodu. Jsem novacek ve webovych aplikacich a v soucasne dobe jsem schopen prispet pouze touto vetickou. V tomto svetle jsem nemel silu si rozchazet prostredi.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants