discord-easy-dashboard icon indicating copy to clipboard operation
discord-easy-dashboard copied to clipboard

Modals

Open TuckBus opened this issue 3 years ago • 8 comments
trafficstars

Adds bootstrap modals that can have the same settings that the base dashboard can. Tested and works fine for me. I requested this in #22 and got bored so I decided to add it.

TuckBus avatar Mar 13 '22 10:03 TuckBus

modals should be done within the theme itself not this way but that's just my 2 cents this will also break 99% of bots and themes if they don't change there code with the way you now have Dashboard.Dashboard when you could of just left that was it was and added Modal without the dashboard infont :)

CreativeCodeCat avatar Mar 16 '22 21:03 CreativeCodeCat

Fair enough, but it shouldn't be too hard to change it, you'd just need to destructure Modal and Dashboard. Just two curly braces should fix it if they have it set up like in the Getting Started example.

TuckBus avatar Mar 21 '22 05:03 TuckBus

Fair enough, but it shouldn't be too hard to change it, you'd just need to destructure Modal and Dashboard. Just two curly braces should fix it if they have it set up like in the Getting Started example.

+1 and people who already use the module will use the previous version until they update it so it's not really a problem I think. I prefer to release breaking updates when the module is small than when many developers are using it.

SimonLeclere avatar Mar 21 '22 06:03 SimonLeclere

Hey, why is the modal section commented in the index.d.ts file ?

SimonLeclere avatar Mar 24 '22 12:03 SimonLeclere

Hey, why is the modal section commented in the index.d.ts file ?

That was a mistake, didn't even realize it was commented out when I was committing. I'll fix it in a few hours when I can get to my computer.

TuckBus avatar Mar 24 '22 15:03 TuckBus

Hey, why is the modal section commented in the index.d.ts file ?

That was a mistake, didn't even realize it was commented out when I was committing. I'll fix it in a few hours when I can get to my computer.

TuckBus avatar Mar 24 '22 15:03 TuckBus

Okay cool, I'll merge it later. Thanks a lot for your contribution

SimonLeclere avatar Mar 24 '22 15:03 SimonLeclere

ups I forgot this one

SimonLeclere avatar Dec 06 '22 09:12 SimonLeclere