-
-
Notifications
You must be signed in to change notification settings - Fork 88
Feat/new component #685
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
algorythmice
wants to merge
34
commits into
PapillonApp:main
Choose a base branch
from
algorythmice:feat/new-component
base: main
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.
+353
−28
Open
Feat/new component #685
Changes from all commits
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
ee35556
feat: implements a new custom menu style instead of the native Androi…
algorythmice 7f984a0
feat: Modification of “MenuView” by the new component “ActionMenu” in…
algorythmice 4f34adf
feat: Modification of “MenuView” by the new component “ActionMenu” in…
algorythmice 160c7da
fix: Add the changes suggested by copilot in PR
algorythmice e697683
fix: Remove id from Averages.tsx because it is no longer necessary an…
algorythmice fab33fb
fix(ui): Replace MenuView with ActionMenu in ChipButton and correct i…
algorythmice c8a2f8f
fix(ui/action-menu): add error handling in ActionMenu.tsx and loads M…
algorythmice 6fec72f
fix(ui/action-menu): Replace any with strict typing for NativeMenuVie…
algorythmice 66e2bd2
fix(ui/action-menu): reusing native types from @react-native-menu
algorythmice c29394e
refactor(ui/action-menu): simplify legacy flag extraction and boolean…
algorythmice 435bf56
refactor(ui/action-menu): simplify state and legacy flag handling
algorythmice 58cde3c
refactor(ui/action-menu): use Stack for item layout
algorythmice 5e31489
refactor(ui/action-menu): change borderBottomColor
algorythmice 6f26487
refactor(ui/action-menu): simplify menu positioning
algorythmice c1a75cf
refactor(ui/action-menu): use textColor for subtitle color
algorythmice bed2b32
refactor(ui/action-menu): replace Unicode chevron/check with Papicons
algorythmice 3bbe1d3
refactor(ui/action-menu): replace Unicode chevron/check with Papicons…
algorythmice 9de87a8
refactor(ui/action-menu): use theme destructive color for destructive…
algorythmice 790cc2a
refactor(ui/action-menu): remove const textColor
algorythmice 563fe6a
refactor(ui/actionmenu): remove hardcoded values and improve readability
algorythmice b248f30
feat(ui/action-menu): support for the “imageColor” parameter
algorythmice 70eceac
feat(app/(modals)/wallpaper): replace MenuView by component ActionMen…
algorythmice 1d802f1
refactor(app/(modals)/wallpaper): remove unused import in wallpaper.tsx
algorythmice 48cf8d3
refactor(app/(modals)/wallpaper): remove unused import in wallpaper.tsx
algorythmice 482bbf9
feat(app/(features)/attendance): replace MenuView by component Action…
algorythmice 26daa16
feat(app/(tabs)/calendar/event/[id]): replace MenuView by component A…
algorythmice b75d0cb
feat(app/(modals)/profile): replace MenuView by component ActionMenu …
algorythmice 3745e25
feat(app/(features)/(news)/specific): replace MenuView by component A…
algorythmice 9ff542c
chore(ui/action-menu): remove redundant NativeMenuView null assignment
algorythmice 79746ea
rfactor(ui/action-menu): use NativeMenuAction instead of MenuAction a…
algorythmice 32bd437
rfactor(ui/action-menu): use NativeMenuComponentProps for component p…
algorythmice 7f82b0f
rfactor(ui/action-menu): simplify colorText logic Corps
algorythmice d383a63
chore(ui/action-menu): use internal logger.warn instead of console.warn
algorythmice 417d7a5
feat(ui/action-menu): handle safe-areas in menu positioning
algorythmice File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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 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 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 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 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 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 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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR description says MenuView usages are replaced by ActionMenu, but there are still multiple
MenuViewimports/usages in the repo (e.g.app/(features)/(news)/specific.tsx,app/(features)/attendance.tsx,app/(modals)/profile.tsx,app/(modals)/wallpaper.tsx,app/(tabs)/calendar/event/[id].tsx). Either update the description/scope, or complete the replacement in this PR/follow-up.