odk-x
ODK-X Instance row control buttons.
ODK-X Instance row control buttons adjusted to fit on one row (edit and delete icons).
ODK-X Survey: Update the buttons, icons and list view of the home screen
Okay . working on it now. Thank you
On Tue, 9 Jul 2024 at 15:28, Ben Green @.***> wrote:
@.**** commented on this pull request.
Hi @juayuohcarineneng19 https://github.com/juayuohcarineneng19 , nice work.
One suggestion I have is to add some padding between these two buttons to reduce the likelihood of tapping the wrong button.
This layout looks better visually, but you could address the potential usability problem as well.
— Reply to this email directly, view it on GitHub https://github.com/odk-x/app-designer/pull/211#pullrequestreview-2166328185, or unsubscribe https://github.com/notifications/unsubscribe-auth/AYBPQRMPLN5K7OMJUPYAWVLZLPQPJAVCNFSM6AAAAABKQX4FE6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDCNRWGMZDQMJYGU . You are receiving this because you were mentioned.Message ID: @.***>
Permit me to nitpick please😅 If it's possible to be adjusted, then great... if not then don't waste too much time on it, you can move on to other things.
Great but I haven't seen a change in the icon ( edit, delete ), the icons had to be changed to the material ones. Perhaps, the delete icon doesn't serve it purpose, it should be changed to the trash icon ( you know what I mean )
@nuilewis Great suggestion there, you just asked what I had in mind
@Redeem-Grimm-Satoshi please this pull request is linked to this design. So I was mainly concern about the design.
Here is the issue on updating every icon to material design guideline
I look forward to working on all of these issues. thank you.
@juayuohcarineneng19 Isn't the icons part of the design? I want the icon changes included in this PR
@Redeem-Grimm-Satoshi please this pull request is linked to this design. So I was mainly concern about the design. Here is the issue on updating every icon to material design guideline I look forward to working on all of these issues. thank you.
@juayuohcarineneng19 Isn't the icons part of the design? I want the icon changes included in this PR
Sure @Redeem-Grimm-Satoshi working on it now.
Permit me to nitpick please😅 If it's possible to be adjusted, then great... if not then don't waste too much time on it, you can move on to other things.
Thank you @nuilewis . Taking these into consideration.
Permit me to nitpick please😅 If it's possible to be adjusted, then great... if not then don't waste too much time on it, you can move on to other things.
Thank you @nuilewis . Taking these into consideration.
@nuilewis @Redeem-Grimm-Satoshi here is what I have now. I am still learning how to use material design icons once I am done I will move to implemting it.
.
Permit me to nitpick please😅 If it's possible to be adjusted, then great... if not then don't waste too much time on it, you can move on to other things.
Thank you @nuilewis . Taking these into consideration.
@nuilewis @Redeem-Grimm-Satoshi here is what I have now. I am still learning how to use material design icons once I am done I will move to implemting it.
.
Much better, good work 👍
Looks fine
@Chinex-Boroja Please check this out for any testing related issues @r0ssing Please check this out, It's ready to be merged
@Redeem-Grimm-Satoshi @nuilewis @maprehensive Please I wish to add modifications made on the icons to this pull request since its not yet merged
Great work! I think these design changes really helps make the application easier to use..
Now that you are working in the 'instances.handlebars' file, is it possible to indent the "Create New Instance" button a little bit so it aligns with the rest of the content? (and maybe remove one of the two horisontal rulers below, so there is only one?)
Great work! I think these design changes really helps make the application easier to use.. Now that you are working in the 'instances.handlebars' file, is it possible to indent the "Create New Instance" button a little bit so it aligns with the rest of the content? (and maybe remove one of the two horisontal rulers below, so there is only one?)
@r0ssing I guess this was addressed in this PR: https://github.com/odk-x/app-designer/pull/208
It will preferable we merge older PR first before the new ones to avoid mix ups
@Redeem-Grimm-Satoshi thanks for pointing this out! In that case I agree, that it will be easier if those are merged first (and backmerged into/updated in this PR)
@Redeem-Grimm-Satoshi thanks for pointing this out! In that case I agree, that it will be easier if those are merged first (and backmerged into/updated in this PR)
Yes sure