Skip to content
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

Dropdown Feature with props for List view #125

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Manav958
Copy link

I have added the the dropdown feature in which the full scrollable list is restricted to 2 elements of the list and dropdown for the other methods
also for backward compatibility so that user can choose between full list view and the new feature i have added prop to the custom tab view file for the same

@manideepk90
Copy link
Contributor

manideepk90 commented Oct 23, 2024

You can apply the theme styles from the Theme that is provided by the user configuration, which in turn derives the theme from the native configuration object.

@Manav958
Copy link
Author

Sir,
Do you mean that the styles that I have written on the top of file
In styles module

I have to remove it and
Use ThemeBasedstyle.res file in the hooks folder to get the same styles?

@manideepk90 manideepk90 linked an issue Oct 23, 2024 that may be closed by this pull request
3 tasks
@manideepk90 manideepk90 removed a link to an issue Oct 23, 2024
3 tasks
@manideepk90
Copy link
Contributor

juspay/hyperswitch#6229

@manideepk90 manideepk90 added the hacktoberfest Issues that are up for grabs for hacktoberfest participants label Oct 23, 2024
@Manav958
Copy link
Author

Happy Diwali Everyone!!

Will the PR be accepted today as it is the last day for hacktoberfest?

Copy link
Contributor

@manideepk90 manideepk90 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested improvements:

  1. This PR is not working in mobile.
  2. The container styles can be replaced by viewStyle and text styles can be replaced by textStyle.
  3. the dlist logic can be improved.
  4. list is not rendering properly.
  5. rescript warning are listed.
  6. virtualization issues with flatlist

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest Issues that are up for grabs for hacktoberfest participants hacktoberfest-accepted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants