search

inoerp / inoERP

inoERP is an OneApp [ Go back-end & Flutter front-end ] based enterprise management system
http://docs.inoerp.com/
Mozilla Public License 2.0
767 stars 675 forks source link

Infinity pop up "Readonly Filed" #35

Open hyunkim9123 opened 8 years ago

hyunkim9123 commented 8 years ago

Getting Infinity pop up "Readonly Filed" when selecting "Group Access ID" field from http://127.0.0.1/inoERP/inoerp/form.php?class_name=user_group_access&mode=9 . I know it is not expected to be focused but should pop up the alert only once.

inoerp commented 8 years ago

It's a read-only field. You can't enter any information. Whenever you click on any read-only field , you will get that message.

On Sat, Jul 16, 2016 at 6:42 PM, Hyun IL (Paul) Kim < notifications@github.com> wrote:

Getting Infinity pop up "Readonly Filed" when selecting "Group Access ID" field from http://127.0.0.1/inoERP/inoerp/form.php?class_name=user_group_access&mode=9 . I know it is not expected to be focused but should pop up the alert only once.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/inoerp/inoERP/issues/35, or mute the thread https://github.com/notifications/unsubscribe-auth/AGPVFBZ4Kv_5RPj8DYsJwMx10GucRlg4ks5qWLWygaJpZM4JN_Nj .

hyunkim9123 commented 8 years ago

I do understand the intended behavior. However, once the pop up is up, I cannot proceed anything on the page before checking "prevent this page from creating additional dialogs" . Also, the other readonly fileds does not have the same alert behavior.

inoerp commented 8 years ago

Tested again. No such issue. The system is not showing that message at all until I click on the read-only field. Share a screenshot of the steps you are following

On Sat, Jul 16, 2016 at 7:00 PM, Hyun IL (Paul) Kim < notifications@github.com> wrote:

I do understand the intended behavior. However, once the pop up is up, I cannot proceed anything on the page before checking "prevent this page from creating additional dialogs" . Also, the other readonly fileds does not have the same alert behavior.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/inoerp/inoERP/issues/35#issuecomment-233125171, or mute the thread https://github.com/notifications/unsubscribe-auth/AGPVFFoNXA_kR5PIUYzxWYlXUbxkqDQ3ks5qWLm2gaJpZM4JN_Nj .

hyunkim9123 commented 8 years ago

Once you click the read-only field, you will see the pop-up appears and cannot do anything before clicking the check box "prevent this page from creating additional dialogs" . Maybe error message next to the read-only textbox is better than alert. For example, if a user selects the read-only field and wants to do move the focus, cannot do anything. It is up to you to fix the behavior or not, but I think it is really inconvenient.

inoerp commented 8 years ago

As I said, send me a screenshot showing the steps you are doing as I can't replicate it. Btw, which version are you using? Genuine issues (not issues due to incorrect user training) will be always fixed. " It is up to you to fix the behavior " was unnecessary. That does not sound good. So, choose your words carefully in a public platform.

On Sat, Jul 16, 2016 at 8:08 PM, Hyun IL (Paul) Kim < notifications@github.com> wrote:

Once you click the read-only field, you will see the pop-up appears and cannot do anything before clicking the check box "prevent this page from creating additional dialogs" . Maybe error message next to the read-only textbox is better than alert. For example, if a user selects the read-only field and wants to do move the focus, cannot do anything. It is up to you to fix the behavior or not, but I think it is really inconvenient.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/inoerp/inoERP/issues/35#issuecomment-233127413, or mute the thread https://github.com/notifications/unsubscribe-auth/AGPVFJDCFx_IMpGUO1fWNDYJ0MQNfYInks5qWMnMgaJpZM4JN_Nj .

hyunkim9123 commented 8 years ago

screenshot-readonly

My apologizes it causes misunderstanding, I did not mean anything other by wording. If you think the the behavior is normal, you can leave it as it is (and close the issue ticket probably) since it is not critical thing. . the version I tested is the most recent one in the master branch. Take a look the screenshot.

hyunkim9123 commented 8 years ago

The thing I feel inconvenient is that the message ups infinitely after I click "ok"