MrGreensWorkshop / MrGreen-JekyllTheme

Mr. Green is a multilingual theme generated with Jekyll and fully compatible with GitHub Pages.
https://jekyll-theme-mrgreen-demo.mrgreensworkshop.com
MIT License
154 stars 294 forks source link

Contact form using Google Forms #143

Closed MrGreensWorkshop closed 1 year ago

MrGreensWorkshop commented 1 year ago

Is this enhancement or feature is related to a problem?

I think we need contact form and this is what I came up with.

Describe the solution you'd like

There are other options to but for personal use, I think Google Forms will be sufficient.

This is the "Contact Form" button added after the about text; it is added to_layouts/about.md.

button

This is the loading text that appears until the Google Form is loaded.

loading

This is the Google Form that has been loaded.

form

Describe alternatives you've considered

https://formspree.io/plans https://formspark.io/pricing

Additional context

How to use?

It is explained in these files. Search for "contact_form" in both these files: _data/owner/[language].yml and _data/conf/main.yml.

MrGreensWorkshop commented 1 year ago

@fandangos Hi, sorry to bother you like this. Could you please take a look at the comments at PR #143, where I mentioned your name, and tell me what you think? (Brazilian Portuguese translations) Thanks in advance.

fandangos commented 1 year ago

No bothering at all, glad to help in whatever I can.

Are we talking about this PR? https://github.com/MrGreensWorkshop/MrGreen-JekyllTheme/pull/144/commits/53e15a12159d5b0fdca2a2e59f4f1a887beba76e

I see just a few lines in portuguese and those are OK to understand, minor tweaks could improve it but it's fine as it is. If you want I can provide a translation to this paragraph: https://github.com/MrGreensWorkshop/MrGreen-JekyllTheme/pull/144/commits/53e15a12159d5b0fdca2a2e59f4f1a887beba76e#diff-95836f60c62131324ab103bf9d6682c429cab8acc6814c780c70acfcf591c8abR73

But it needs to get merged first so I know how's the final text that needs translation.

Let me know what I can do to help and I'll do it.

MrGreensWorkshop commented 1 year ago

@fandangos Thank you, alright I think I will merge this so you can push a PR to the repo.

These are readme files

English https://github.com/MrGreensWorkshop/MrGreen-JekyllTheme/blob/2e396c74dc70c03844b3e2bc3edb05569709dfbe/README.md?plain=1#L59

Brazilian Portuguese https://github.com/MrGreensWorkshop/MrGreen-JekyllTheme/blob/2e396c74dc70c03844b3e2bc3edb05569709dfbe/README-pt.md?plain=1#L59

https://github.com/MrGreensWorkshop/MrGreen-JekyllTheme/blob/2e396c74dc70c03844b3e2bc3edb05569709dfbe/pt/_posts/_README-pt.md?plain=1#L59

These are language files

English https://github.com/MrGreensWorkshop/MrGreen-JekyllTheme/blob/2e396c74dc70c03844b3e2bc3edb05569709dfbe/_data/lang/en.yml#L175-L177

Brazilian Portuguese https://github.com/MrGreensWorkshop/MrGreen-JekyllTheme/blob/2e396c74dc70c03844b3e2bc3edb05569709dfbe/_data/lang/pt.yml#L175-L177

fandangos commented 1 year ago

Great! Having the directions to what needs to be translated makes things much faster.

My family is here for dinner, can't do it today. I'll do it tomorrow after breakfast.

MrGreensWorkshop commented 1 year ago

No problem at all, whenever you feel okay.

MrGreensWorkshop commented 1 year ago

This issue is finished, as the feature was made in PR #144 and the Brazilian Portuguese translations were checked and improved by @fandangos in PR #145.

@fandangos Thank you again for your time.