Skip to content

Fix something wrong in index.html template#58

Open
issei-m wants to merge 2 commits into
Azure-Samples:masterfrom
issei-m:patch-1
Open

Fix something wrong in index.html template#58
issei-m wants to merge 2 commits into
Azure-Samples:masterfrom
issei-m:patch-1

Conversation

@issei-m

@issei-m issei-m commented Aug 3, 2020

Copy link
Copy Markdown
  • Fix some syntax errors of HTML
  • Remove useless JavaScript code
  • Fix indent

- Fix some syntax errors of HTML
- Remove useless JavaScript code
- Fix indent
@issei-m

issei-m commented Sep 10, 2020

Copy link
Copy Markdown
Author

ping @zr-msft since you seem to be a person who made the latest merge commit.
If you had the time to review it, please see it.
Regards.

@Point-SimoSiren

Copy link
Copy Markdown

ping @issei-m Hello, this interesting tutorial. I tried to change names other than purple and blue, and app doesnt work anymore. I wonder is it really so that app can handle only dog, cat and blue, purple ? Another question is that why you have left button onClick(send) handlers if u removed the script? I have seen input type submit but is this send similar inside form?

@issei-m

issei-m commented Oct 2, 2020

Copy link
Copy Markdown
Author

I'll take it on this weekend :)

@issei-m

issei-m commented Oct 3, 2020

Copy link
Copy Markdown
Author

@Point-SimoSiren I could rename the caption of the buttons by updating config_file.cfg where the captions of them are defined, so try it on.
image

And leaving the onclick handlers was just my mistake, I fixed it, Thanks pointing out it.

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.

2 participants