dash-daq icon indicating copy to clipboard operation
dash-daq copied to clipboard

Added left and right options for labelPosition in toggles.

Open Akronix opened this issue 5 years ago • 10 comments

This is a straightforward intent to fix #29.

I have noted that there are more components which use labelPosition, I could also change their react props if you find this PR useful.

Let me know if there's anything else I'm missing.

Akronix avatar Feb 15 '19 17:02 Akronix

ping @shammamah

Akronix avatar Feb 19 '19 16:02 Akronix

@Akronix Would you mind resetting the irrelevant files (i.e., anything that doesn't have to do with your specific changes) to the versions on master? There are lots of diffs in the PR that amount to pretty-printing or documentation. Would make it easier to review :)

shammamah-zz avatar Feb 19 '19 22:02 shammamah-zz

Let's see now :)

Akronix avatar Feb 20 '19 17:02 Akronix

Not sure if it worked -- if you go into "Files changed", you'll see that the documentation files still have diffs. I think it's because there are some hooks in place for this repository that auto-gen the documentation at each commit. I'll just build the package and test your code in the meantime!

shammamah-zz avatar Feb 20 '19 21:02 shammamah-zz

Hey @shammamah, this looks like a great addition to the toggles. Any update on merging this pull request? Thanks!

Shourai avatar Jul 30 '19 09:07 Shourai

@Akronix @Shourai Yes, this is a great addition! Apologies for not having gotten to it earlier.

@Akronix would you be able to merge in the latest master into your fork? We've made quite a few changes, including ones that will fix this auto-doc-generation issue.

shammamah-zz avatar Jul 30 '19 19:07 shammamah-zz

@Akronix @Shourai Yes, this is a great addition! Apologies for not having gotten to it earlier.

@Akronix would you be able to merge in the latest master into your fork? We've made quite a few changes, including ones that will fix this auto-doc-generation issue.

I'm pretty much AFK until 15th of August. is it fine if I do it after that? Please ping me if I forget about it after then or even on the same 15th :smiley:

Akronix avatar Aug 03 '19 18:08 Akronix

Sure no worries!

shammamah-zz avatar Aug 05 '19 13:08 shammamah-zz

@Akronix @shammamah Any update on this one? Or is there any alternative that is currently available

kjsr7 avatar Apr 22 '21 05:04 kjsr7

@Akronix @shammamah-zz can we have an update on this? I couldn't find a way to have a label left of any toggle... it seems this PR is dead. I don't mind trying to reimplement it if @Akronix doesn't wish to merge it. In that case maybe it should be closed?

RemDelaporteMathurin avatar Oct 05 '22 15:10 RemDelaporteMathurin

Sorry, I'm closing this because of lack of time and because currently I'm working in a very different area. Though, anyone can use whatever they want from my code.

Akronix avatar Nov 28 '22 17:11 Akronix