-
Notifications
You must be signed in to change notification settings - Fork 5
Task registration screen #93
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
Open
LucasOliveiraSimao
wants to merge
68
commits into
develop
Choose a base branch
from
task-registration-screen
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This commit refactors the `InputText` composable:
- Updates padding from `16.sdp` to `16.dp`.
- Modifies `titleText` styling:
- Removes explicit `start` and `end` padding.
- Changes text style from `MaterialTheme.typography.bodyMedium` to `MaterialTheme.typography.titleMedium`.
- Adjusts `placeholderText` styling:
- Sets text color to `ColorCustom.color_placeholder`.
- Changes text style from a custom `TextStyle` to `MaterialTheme.typography.bodyMedium`.
This commit introduces the `ScreenRegisterTask` composable, a new screen for registering tasks within the application.
Key changes include:
- Addition of `ScreenRegisterTask.kt` file.
- Implementation of the `ScreenRegisterTask` composable function, which includes:
- A `ScaffoldCustom` for the basic screen structure with a top app bar titled "Nova tarefa" (New Task) and bottom navigation.
- A `LazyColumn` to display various input fields and components for task registration.
- `InputText` for the task name.
- `TextFieldCustom` for the task description and an observation field.
- `GroupSelectableButton` (functionality not fully detailed in the diff).
- `PetFilterList` to select a pet associated with the task.
- `TransactionTypeSelector` to choose between recurrent and one-off task types.
- A `@Preview` function `ScreenRegisterTaskPreview` is included for UI development and testing.
…es vetorizados nos tipos de tarefas
# Conflicts: # petJournal/app/src/main/res/values/strings.xml
This commit introduces the `ScreenRegisterTask` composable, a new screen for registering tasks within the application.
Key changes include:
- Addition of `ScreenRegisterTask.kt` file.
- Implementation of the `ScreenRegisterTask` composable function, which includes:
- A `ScaffoldCustom` for the basic screen structure with a top app bar titled "Nova tarefa" (New Task) and bottom navigation.
- A `LazyColumn` to display various input fields and components for task registration.
- `InputText` for the task name.
- `TextFieldCustom` for the task description and an observation field.
- `GroupSelectableButton` (functionality not fully detailed in the diff).
- `PetFilterList` to select a pet associated with the task.
- `TransactionTypeSelector` to choose between recurrent and one-off task types.
- A `@Preview` function `ScreenRegisterTaskPreview` is included for UI development and testing.
…nal/petjournal.android into task-registration-screen
- Update ActiveMonthsComponent with better styling - Enhance DateInputText component layout - Refactor DropDown component structure - Remove title text from DualActionButton for cleaner design - Improve InputText component with better padding and layout - Update NavigationBarItems styling These changes enhance the overall UI consistency and user experience.
- Change 'Cadastro Pet' to 'Cadastro' for better UX - Simplifies the registration screen title
- Replace missing ColorCustom properties with error_color as temporary solution - Add TODO comments for proper color implementation - Ensures dialog renders correctly while proper colors are being defined This is a temporary fix that should be replaced once proper color definitions are available.
- Refactor layout structure with better spacing and padding - Add navigationBarsPadding for better screen adaptation - Improve LazyColumn arrangement with consistent spacing - Add individual title labels for DualActionButton components - Create comprehensive preview components for all form elements - Enhance component styling and MaterialTheme usage - Remove redundant padding and improve component organization This improves the overall user experience and developer preview capabilities.
- Update NavHostElements route from 'pets/petList' to 'pets/petListScreen' - Add navigation functionality to PetItemMore component - Connect pet list screen to register pet screen navigation - Remove commented navigation route This ensures proper navigation flow between pet list and registration screens.
- Fix parameter order in InputText test components - Ensure consistent parameter ordering for better test reliability - Update test structure to match component interface changes This improves test consistency and prevents parameter order issues.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Task registration screen was created and UI tests were implemented