-
Notifications
You must be signed in to change notification settings - Fork 1
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
Feat/inject header right #18
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Claude Arseneault <[email protected]>
Signed-off-by: Claude Arseneault <[email protected]>
const defaultStackOptions = useDefaultStackOptions(theme) | ||
const [Options] = useServices([TOKENS.COMPONENT_OPTIONS]) |
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.
renommer Options par stackOptions
|
||
const HomeStack: React.FC = () => { | ||
const Stack = createStackNavigator<HomeStackParams>() | ||
const theme = useTheme() | ||
const { t } = useTranslation() | ||
const [store] = useStore() | ||
const defaultStackOptions = useDefaultStackOptions(theme) | ||
const [Options] = useServices([TOKENS.COMPONENT_OPTIONS]) |
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.
Idem pour renommer
proofDetailsStackOptions: { headerRight: HeaderRightHome }, | ||
proofReqUsageHistoryStackOptions: { headerRight: HeaderRightHome }, | ||
credentialsStackOptions: { headerLeft: SettingsMenu }, | ||
globalConnectionStackOptions: { headerRight: HeaderRightHome, headerLeft: () => null }, |
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.
Pourquoi mettre le headerLeft à () => null ? Peut-être juste le retirer serait correct.
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.
Il était comme ça au départ dans la stack, je me suis dit qu'il y avait probablement une raison
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.
Voir les commentaires
Signed-off-by: Claude Arseneault <[email protected]>
Signed-off-by: Claude Arseneault <[email protected]>
Signed-off-by: Claude Arseneault <[email protected]>
Signed-off-by: fc-santos <[email protected]>
Signed-off-by: Claude Arseneault <[email protected]>
Signed-off-by: Claude Arseneault <[email protected]>
Signed-off-by: ClaudeArs <[email protected]>
Signed-off-by: ClaudeArs <[email protected]>
Summary of Changes
Replace this text with a high-level summary of the changes included in this PR.
Related Issues
Please reference here any issue #'s that are relevant to this PR, or simply enter "N/A" if this PR does not relate to any existing issues.
Pull Request Checklist
Tick all boxes below to demonstrate that you have completed the respective task. If the item does not apply to your this PR check it anyway to make it apparent that there's nothing to do.
Signed-off-by
line (we use the DCO GitHub app to enforce this);If you have any questions to any of the points above, just submit and ask! This checklist is here to help you, not to deter you from contributing!
Pro Tip 🤓
PR template adapted from the Python attrs project.