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

Fix #1024: Fix Tom-select in dark mode #1098

Merged
merged 1 commit into from
Apr 11, 2022
Merged

Conversation

rjd22
Copy link
Collaborator

@rjd22 rjd22 commented Mar 24, 2022

No description provided.

@vercel
Copy link

vercel bot commented Mar 24, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/tabler-ui/tabler/9kbNWmEizbr1KNEbX3LSFdksKuao
✅ Preview: https://tabler-git-fix-tom-select-dark-mode-tabler-ui.vercel.app

@kevinpapst
Copy link
Collaborator

The second commit fixed the visibility issue 👍

Could you add an optgroup item to one of the dropdowns on the form-elements.html page?
I had issues with them in my app

@rjd22
Copy link
Collaborator Author

rjd22 commented Mar 24, 2022

@kevinpapst if you can make an example for me I will add it to the docs.

@kevinpapst
Copy link
Collaborator

kevinpapst commented Mar 24, 2022

This is an example with fake data from my app (so probably values and classes need adjustment):

<select class="selectpicker form-select tomselected ts-hidden-accessible">
	<option value=""></option>
	<optgroup label="Organic responsive artificialintelligence">
		<option value="1281">aggregate magnetic supply-chains</option>
		<option value="1274">cultivate intuitive synergies</option>
		<option value="1282">embrace enterprise webservices</option>
		<option value="1268">envisioneer out-of-the-box infrastructures</option>
		<option value="1277">extend dot-com relationships</option>
		<option value="1275">integrate ubiquitous ROI</option>
		<option value="1280">matrix seamless niches</option>
		<option value="1270">optimize distributed e-tailers</option>
		<option value="1276">seize impactful e-business</option>
		<option value="1271">synergize vertical architectures</option>
		<option value="1278">transform turn-key e-business</option>
		<option value="1272">whiteboard value-added e-markets</option>
	</optgroup>
	<option value="2864">architect 24/365 users</option>
	<option value="2867">cultivate visionary models</option>
	<option value="2856">deploy intuitive relationships</option>
	<option value="2855">empower innovative metrics</option>
	<option value="2865">enable holistic e-business</option>
	<option value="2868">engage extensible users</option>
	<option value="2849">enhance open-source vortals</option>
	<option value="2845">enhance wireless eyeballs</option>
	<option value="2857">implement viral ROI</option>
	<option value="2847">innovate B2B portals</option>
	<option value="2844">matrix holistic technologies</option>
	<option value="2863">recontextualize granular users</option>
	<option value="2843">reintermediate 24/7 e-tailers</option>
	<option value="2851">reinvent wireless e-business</option>
	<option value="2848">revolutionize enterprise interfaces</option>
	<option value="2852">revolutionize wireless supply-chains</option>
	<option value="2859">streamline synergistic users</option>
	<option value="2860">syndicate best-of-breed e-services</option>
	<option value="2853">unleash cutting-edge networks</option>
	<option value="2861">utilize efficient infomediaries</option>
	<option value="1269">enable dot-com infrastructures
	</option>
</select>

You could probably duplicate the optgroup for test purposes.

@rjd22
Copy link
Collaborator Author

rjd22 commented Apr 11, 2022

@kevinpapst I will merge this first and look into outgroups later when I have some more time.

@rjd22 rjd22 merged commit d4097c4 into main Apr 11, 2022
@rjd22 rjd22 deleted the fix-tom-select-dark-mode branch April 11, 2022 07:11
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.

None yet

2 participants