-
Notifications
You must be signed in to change notification settings - Fork 0
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
Create new record view #5
Comments
As a user I want to enter information such as the description, the monetary value and date or time for every entry. We have to figure out how we will support this but at least we will need inputs for the description (text), price (numeric value where the keyboard should be a keypad and there should be a dropdown for the currency) and the date (use DatePicker). Dont worry about saving. We just need the layouts in place, the functional will be added later. Create I believe we currently have the "camera" set up here, but let's just have the "new record" click fire a "CreateRecord" event which may fire up the camera first of the form depending on a setting which we will define later. |
what about a mockup ? |
We go for native experience UI-wise, so own designs 😉. I'm drafting mockups at the moment. |
ok cool , |
I've already set up Just focus on getting the form to look and feel like the mockups. The functionality will be added later when we have the dispatcher and other plumbing for actions and stores set up 😉 TODO
|
Hi @vidbina ! Could you help me with icons for this view? do you have a source assets? Thanks! |
@AlinaChernenko can you push it we can work on it together . |
@vidbina - I will update iOS with DatePicket and Camera on next commit. |
@zirconias sure, it's in the last commit of the setup/new-record-form branch. |
@vidbina Hi David! Need your help =) |
Hey @AlinaChernenko, I did something but it is somewhat ugly still. I assumed that for every screen size offsetting the FAB (floating action button) by the height of the screen |
Great work @AlinaChernenko 👍 . Just the finishing touches left.
|
For Android we need the following:
|
@vidbina sure, will update everything. Just one thing with warnings. I was trying to get rid of it and it seems other guys have the issue facebook/react-native#4547 even though the code is appropriate. I used temporary solution and ignore Warning: Failed propType for now. Hope that is fine. |
OK. I'll keep that in mind. It is fine. |
Upon clicking the "add record" button (see FAB in Android, see ➕ in iOS) a camera view should pop up which allows one to snap a picture followed by a view which allows one to specify a price and description.
The text was updated successfully, but these errors were encountered: