Skip to content

fix: content not visible on js version#731

Closed
vonovak wants to merge 1 commit intoreact-native-segmented-control:masterfrom
vonovak:fix/js-not-visible
Closed

fix: content not visible on js version#731
vonovak wants to merge 1 commit intoreact-native-segmented-control:masterfrom
vonovak:fix/js-not-visible

Conversation

@vonovak
Copy link
Copy Markdown
Contributor

@vonovak vonovak commented Jun 20, 2023

Overview

This fixes the appearance of the JS version of the segmented control

screenshots

before

Screenshot_1687268903

after

Screenshot_1687269144

Test Plan

I'm not able to test because I'm not able to start up the example app in this repo, but tested this elsewhere

@vonovak vonovak requested a review from Naturalclar as a code owner June 20, 2023 13:54
@Naturalclar
Copy link
Copy Markdown
Member

currently organizing stuff around ci and example project.
will look at the pr afterwards 🙏

@Saadnajmi
Copy link
Copy Markdown
Contributor

Hey @Naturalclar , I noticed the datetimepicker repo still has a patch to add this fix to segmented-control in their example app. Would you mind if I remade this PR to perhaps remove the need for the patch?

@Naturalclar
Copy link
Copy Markdown
Member

@Saadnajmi sure! That would help out alot!
Sorry for not being able to keep up with the maintenance 🙏

@Saadnajmi
Copy link
Copy Markdown
Contributor

Saadnajmi commented Apr 18, 2024

@Saadnajmi sure! That would help out alot! Sorry for not being able to keep up with the maintenance 🙏

No worries, you maintain a lot! (How much stuff exactly? 😅) Happy to help out when/where I can, even if entirely motivated by code I'm currently touching haha. #848

@Naturalclar
Copy link
Copy Markdown
Member

even if entirely motivated by code I'm currently touching haha

Thats like the only time im motivated to touch codes as well haha
Unfortunately i havent been in need for this component in a while..

@Saadnajmi
Copy link
Copy Markdown
Contributor

@vonovak looks like you can close this now. I'll update my PR to datetiimepicker to remove the patch and pull the new version

@Naturalclar
Copy link
Copy Markdown
Member

released in v2.5.2 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants