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

Add info - creating ajax request in the browser #990

Merged
merged 2 commits into from
Apr 7, 2023
Merged

Add info - creating ajax request in the browser #990

merged 2 commits into from
Apr 7, 2023

Conversation

mildabre
Copy link
Contributor

@mildabre mildabre commented Apr 5, 2023

Hi,

I send a proposal to complete info about the http ajax request. The introduction paragraph talks about an ajax http request on the server, but newcomers need a brief input on how an ajax request is actually created - that's what the paragraph should ideally start with. It is also appropriate to give an Ajax request as such a separate title.

Hi,

I send a proposal to complete info about the http ajax request. The introduction paragraph talks about an ajax http request on the server, but newcomers need a brief input on how an ajax request is actually created - that's what the paragraph should ideally start with. It is also appropriate to give an Ajax request as such a separate title.
Ajaxový požadavek
=================

V prohlíčeči vytvoření ajaxového http požadavku zajistí javascriptová knihovna Naja. Aby se z obyčejného http požadavku (signál odkazu, odeslání formuláře, ...) vytvořil ajaxový požadavek, je potřeba označit příslušný odkaz (formulář, odesílací tlačítko formuláře):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Odkaz na Naja která je na stránce odkázaná o kousek níž?

Copy link
Contributor Author

@mildabre mildabre Apr 5, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Já sám jsem se zde na chvilku zaseknul, protože jsem se nedočetl jak se ajaxový požadavek vytvoří, ale napadlo mě si to v odkazu na Naja najít. Syn, který byl na školení Nette a znal odtamtud a href=".." class="ajax" ale nevěděl jak odeslat ajaxově formulář, hledal to v dokumentaci, ale nenapadlo ho jít do odkazu Naja. Sám na to nepřišel, podobně i druhý syn se také zasekával na podobných drobnostech.

Přijde mě logické, když se začne mluvit o zpracování ajaxového požadavku dát před to dvě-tři věty jak ajaxový požadavek vznikne. Aby v odstavci v úvodu byly tyto informace pohromadě.

Druhou možností je přesunout vzorek kódu který jsem poslal do sekce o knihovně Naja.

Copy link
Contributor

@mabar mabar Apr 5, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Já tím myslel, že by to chtělo prolinkovat vytvořením odkazu na kapitolu o Naja. Důvod chápu.

application/cs/ajax.texy Outdated Show resolved Hide resolved
@dg
Copy link
Member

dg commented Apr 6, 2023

U toho formuláře to funguje v obou případech, tj když je to na form nebo na tlačítku?

Co-authored-by: Marek Bartoš <bartos.developer152@gmail.com>
@dg dg merged commit dd0353b into nette:master Apr 7, 2023
dg pushed a commit that referenced this pull request Apr 8, 2023
dg pushed a commit that referenced this pull request Apr 8, 2023
dg pushed a commit that referenced this pull request Apr 9, 2023
dg pushed a commit that referenced this pull request Apr 9, 2023
dg pushed a commit that referenced this pull request Apr 9, 2023
dg pushed a commit that referenced this pull request Apr 9, 2023
dg pushed a commit that referenced this pull request Apr 9, 2023
dg pushed a commit that referenced this pull request Apr 9, 2023
dg pushed a commit that referenced this pull request Apr 9, 2023
dg pushed a commit that referenced this pull request Apr 18, 2023
dg pushed a commit that referenced this pull request Apr 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants