Create a copy button about geeksay

[dk3775]

@abitsalihu you are good to go I am assigning on this!

[abitsalihu]

@dk3775 I would like to work on this issue :)

[abitsalihu]

@dk3775 I finished this issue, but I think I made a mistake to what should be copied. I made the copy button to copy the random quote that is generated or should it copy the : Random Translation: (this part that is inside here) ?

If you could tell me if i did it correctly or I should make changes so I can move on with the PR. :)

Below is the video of how I made it:

Screen.Recording.2022-09-19.at.14.00.39.mov

[abitsalihu]

@dk3775 I read the Issue again, and I think I understand what needed to be copied (the translated quote that is generated from the random quote)!

Below is a video of how the "copy button" works now:
Please tell me if this is correct so I can go ahead and make a PR.

Screen.Recording.2022-09-19.at.14.20.02.mov

[dk3775]

@abitsalihu bro actually we had to copy whatever text is there in translated textbox, I guess just changing the js object will do the needful.

Hope you understood that, else just ping me again anytime!!

[abitsalihu]

@dk3775 I already made a PR yesterday, can you please check it out and tell me what's wrong.
There is a video attached with it to see how it works!

[dk3775]

@abitsalihu ooh my bad I didn't check it correctly it's working perfectly fine, thanks!

[abitsalihu]

@dk3775 please check the latest commit I did on the PR. :)

[dk3775]

Hello @abitsalihu you did great work adding that button,
Still I think we need better styling to the button.

[abitsalihu]

@abitsalihu thank you, I styled it the same as the random button as I was told so. :)

[dk3775]

@abitsalihu I know but now think it does not go with the flow :-|

[RohanTheProgrammer]

Hi @dk3775 I am working on the copy button trying to make it a little bit better. I am improving the alert and trying to make the copy button look more natural to the page.

[aharna]

hey! I would like to work on this issue

[RohanTheProgrammer]

Hi @aharna
Well I have made a version of copy button and made a PR. Still both of us will have to wait for @dk3775 for his reponse.

[dk3775]

Hello @RohanTheProgrammer the button looks fine in desktop view, but mobile view needs to be fixed. Just make it dynamic.

Anyone else can also contribute to PR #369 on this issue.

[dk3775]

This issue is solved by #369 from @RohanTheProgrammer. And is now closed.