Skip to content

Conversation

@GeoSot
Copy link
Member

@GeoSot GeoSot commented Apr 10, 2021

Use one helper for all list next|previous functionality and use it on dropdown.js & carousel.js

@GeoSot GeoSot requested a review from alpadev April 11, 2021 19:10
@GeoSot GeoSot force-pushed the gs-list_picker-helper branch 2 times, most recently from a1b4e6e to 55006cf Compare April 15, 2021 08:38
@GeoSot GeoSot marked this pull request as ready for review April 15, 2021 08:41
@GeoSot GeoSot requested a review from a team as a code owner April 15, 2021 08:41
@XhmikosR XhmikosR changed the title Index.js: make 'pickFromList' helper index.js: add pickFromList helper Apr 15, 2021
@GeoSot GeoSot force-pushed the gs-list_picker-helper branch from 55006cf to 8016046 Compare April 19, 2021 08:29
@GeoSot GeoSot force-pushed the gs-list_picker-helper branch 2 times, most recently from f83badd to a53cb79 Compare May 10, 2021 22:03
@GeoSot GeoSot force-pushed the gs-list_picker-helper branch from b74e09c to c27a1c7 Compare May 13, 2021 20:40
@rohit2sharma95 rohit2sharma95 self-requested a review May 14, 2021 12:53
Copy link
Contributor

@rohit2sharma95 rohit2sharma95 left a comment

Choose a reason for hiding this comment

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

Better pickFromList => getNextActiveElement 🤔

Good work @GeoSot
🛠️ 🧰

@rohit2sharma95 rohit2sharma95 changed the title index.js: add pickFromList helper Utils: add getNextActiveElement helper function May 14, 2021
@GeoSot GeoSot force-pushed the gs-list_picker-helper branch from ba76e09 to 2ff9ede Compare May 18, 2021 22:01
@GeoSot GeoSot force-pushed the gs-list_picker-helper branch from 2ff9ede to 0807a55 Compare May 18, 2021 22:12
@GeoSot GeoSot merged commit df72a21 into main May 18, 2021
@GeoSot GeoSot deleted the gs-list_picker-helper branch May 18, 2021 22:23
marvin-hinkley-vortx pushed a commit to Vortx-Inc/bootstrap that referenced this pull request May 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants