search

adshayanB / jobMaps

3 stars 0 forks source link

Application List UX #112

Closed saymaakhan closed 3 years ago

Jaishreebala commented 3 years ago

https://www.figma.com/file/DeYj4unPyPC5lB6IlsTYU1/Finance-Tracker-team-library?node-id=312%3A2 let me know any changes

Screen Shot 2021-07-24 at 2 43 16 PM

or if u dont feel like openning figma its up there ^ lmk anything to change i havent updated the filters stuff in it yet tho

saymaakhan commented 3 years ago

BEAUTIFUL

saymaakhan commented 3 years ago

I think we may need a "..." for deleting or editing a modal, or some type of button to prompt the modal if they need to change something, but otherwise it looks great:)

Jaishreebala commented 3 years ago

I think we may need a "..." for deleting or editing a modal, or some type of button to prompt the modal if they need to change something, but otherwise it looks great:)

I was thinking we just let them click on the field to update? bc its easier to use from a user perspective? idk and for delete have a button? idkkk im oki w anything tho

saymaakhan commented 3 years ago

I think we may need a "..." for deleting or editing a modal, or some type of button to prompt the modal if they need to change something, but otherwise it looks great:)

I was thinking we just let them click on the field to update? bc its easier to use from a user perspective? idk

oh does this mean that the only modal needed is for adding a new app?

Jaishreebala commented 3 years ago

I think we may need a "..." for deleting or editing a modal, or some type of button to prompt the modal if they need to change something, but otherwise it looks great:)

I was thinking we just let them click on the field to update? bc its easier to use from a user perspective? idk

oh does this mean that the only modal needed is for adding a new app?

yeah, i think itll also be easier to implement from a front end perspective too? i always find modals annoyig haha

adshayanB commented 3 years ago

To add I kinda like everything lowercase

adshayanB commented 3 years ago

Also Calendar spelling wrong?

Jaishreebala commented 3 years ago

To add I kinda like everything lowercase

yup, in the acc app everything will be lower case (which is set in the backend i think?)

Jaishreebala commented 3 years ago

Also Calendar spelling wrong?

TY i always spell it wrong lmaooo

Jaishreebala commented 3 years ago

Okay I added a small hover functionality and a delete button, fixed spelling , if theres any other changes lmk

adshayanB commented 3 years ago

To add I kinda like everything lowercase

yup, in the acc app everything will be lower case (which is set in the backend i think?)

Well what you need to acc is to do .lowerCase ot sum and send the body after it