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

Feat : UIDT impmporovements #2026

Merged
merged 19 commits into from
May 16, 2022
Merged

Feat : UIDT impmporovements #2026

merged 19 commits into from
May 16, 2022

Conversation

pranavxc
Copy link
Member

@pranavxc pranavxc commented May 12, 2022

Change Summary

UI datatype improvements and customization.

Change type

  • Rating type customisation and improvements
  • Checkbox type customisation
  • Email/URL - optional validation and rendering as link

Test/ Verification

Rating

Screenshot 2022-05-12 at 2 07 59 PM
Screenshot 2022-05-12 at 2 08 13 PM

Checkbox

Screenshot 2022-05-12 at 2 09 34 PM
Screenshot 2022-05-12 at 2 10 32 PM

Copy link
Member

@wingkwong wingkwong left a comment

Choose a reason for hiding this comment

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

Just one doubt.

If I set max rating to 8, select 8, then change the max rating to 4. What is the value supposed to be? The current behaviour is taking the previous value which is 8 but you can only select 4 for others. May need to discuss whether we should take the upper bound or just use the existing value.

@dstala
Copy link
Member

dstala commented May 13, 2022

  1. Gallery view, we do not show 'rating' & show numbers instead
    Screenshot 2022-05-13 at 3 55 00 PM
  2. v1, if numbers were high (rating did not have any restriction in v1) - it gets mapped to default 5* in v2
  3. add rating column to default table. add new row (sqlite root db, sqlite project)
    Screenshot 2022-05-13 at 4 03 03 PM

@pranavxc pranavxc force-pushed the feat/uidt-improvements branch 2 times, most recently from 02ba07e to 9fc607b Compare May 15, 2022 14:34
Signed-off-by: Pranav C <pranavxc@gmail.com>
Signed-off-by: Pranav C <pranavxc@gmail.com>
Signed-off-by: Pranav C <pranavxc@gmail.com>
Signed-off-by: Pranav C <pranavxc@gmail.com>
Signed-off-by: Pranav C <pranavxc@gmail.com>
Signed-off-by: Pranav C <pranavxc@gmail.com>
Signed-off-by: Pranav C <pranavxc@gmail.com>
Signed-off-by: Pranav C <pranavxc@gmail.com>
Signed-off-by: Pranav C <pranavxc@gmail.com>
Signed-off-by: Pranav C <pranavxc@gmail.com>
Signed-off-by: Pranav C <pranavxc@gmail.com>
Signed-off-by: Pranav C <pranavxc@gmail.com>
Signed-off-by: Pranav C <pranavxc@gmail.com>
Signed-off-by: Pranav C <pranavxc@gmail.com>
re #2036

Signed-off-by: Pranav C <pranavxc@gmail.com>
re #2038

Signed-off-by: Pranav C <pranavxc@gmail.com>
Signed-off-by: Pranav C <pranavxc@gmail.com>
Signed-off-by: Pranav C <pranavxc@gmail.com>
Signed-off-by: Pranav C <pranavxc@gmail.com>
@pranavxc pranavxc merged commit 8d33fa9 into develop May 16, 2022
@pranavxc pranavxc deleted the feat/uidt-improvements branch May 16, 2022 10:30
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.

3 participants