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 placeholder for Feedback fields and Persian Localization #70

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

azarbara
Copy link

…ion,Country,PostalCode,Telepone,Message fields in form and field setting, Also add Persian (fa-IR) localized files

The placeholder is a main part of any textbox so I decided to add this to inputs.

Changes made

I add placeholder field after each input in "Form and Field Setting" section
save the values in ModuleSetting table
image
and finally show them in feedback form
image

also I add Persian resx file to localize to persian (fa-IR)

…ion,Country,PostalCode,Telepone,Message fields in form and field setting, Also add Persian (fa-IR) localized files
Copy link

@uzmannazari uzmannazari left a comment

Choose a reason for hiding this comment

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

It seems great

@WillStrohl
Copy link
Member

This looks like a solid update, but I'm not the module "owner" at this time. I'll let @valadas or someone else chime in further.

@azarbara
Copy link
Author

azarbara commented May 3, 2023

@valadas
could you please check this PR!

@valadas
Copy link
Member

valadas commented May 3, 2023

It is on my list for open-source stuff. That list is a bit large and I am preparing for DNN Connect the next few weeks of my free time. It looked ok from a quick look but wanted to take the time to look at this properly.

@uzmannazari
Copy link

could you please publish this pull request mr valadas?

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

Successfully merging this pull request may close these issues.

4 participants