Implemented suggested changes for the Settings

locary-ph / merchant-dashboard

Merchant dashboard for Locary.

https://dashboard.locary.ph

MIT License

0 stars 0 forks source link

Implemented suggested changes for the Settings #123

Closed bojonvi closed 3 years ago

bojonvi commented 3 years ago

Fixes #122 Shop Settings:

Added placeholders

Payment Settings:

Removed Arrows in Number Input
Added Place Holders
Changed the spelling of E-Wallet into eWallet

Delivery Settings:

Added Placeholders
Changed Subtexts

[ ] PR is descriptively titled 📑 and links the original issue above 🔗
[x] screenshots/GIFs are attached :paperclip: in case of UI changes

github-actions[bot] commented 3 years ago

🚀 Netlify deployed locary-alpha as draft

https://61461273f5df65c875e3d777--locary-alpha.netlify.app

bojonvi commented 3 years ago

@jcads, should we remove the cancel button or are we gonna add a function where it will redirect the user back to the home dashboard? Because at the moment, it doesn't do anything.

jcads commented 3 years ago

@jcads, should we remove the cancel button or are we gonna add a function where it will redirect the user back to the home dashboard? Because at the moment, it doesn't do anything.

I think a history.goBack() will suffice here.

github-actions[bot] commented 3 years ago

🚀 Netlify deployed locary-alpha as draft

https://6152005285d7e6b3520c0e60--locary-alpha.netlify.app

github-actions[bot] commented 3 years ago

🚀 Netlify deployed locary-alpha as draft

https://6155a38041ca6502095e553f--locary-alpha.netlify.app