Fixed the size proportions of Shuffle Button

0x00zer0day commented 9 months ago

Related Issue

Closes: #114

Describe the changes you've made

Changed the Height and the Width of Shuffle button from 25px to 30px

Type of change

What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

Ran an NPM server locally and modified the changes real-time

Checklist:

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] Any dependent changes have been merged and published in downstream modules.

Screenshot of the changes made: 2023-10-02T12:32:42,519862604+05:30

vercel[bot] commented 9 months ago

Someone is attempting to deploy a commit to a Personal Account owned by @Utkarshn10 on Vercel.

@Utkarshn10 first needs to authorize it.

Utkarshn10 commented 9 months ago

Hey @0x00zer0day can you also add the screenshot of the page, rest LGTM. Thanks

0x00zer0day commented 9 months ago

Hey @0x00zer0day can you also add the screenshot of the page, rest LGTM. Thanks

Screenshot of the button right? Sure just a sec

0x00zer0day commented 9 months ago

@Utkarshn10 Are the changes correct?

Utkarshn10 commented 9 months ago

Hey @0x00zer0day can you also add the screenshot of the page, rest LGTM. Thanks

Screenshot of the button right? Sure just a sec

Screenshot of the whole page.

Utkarshn10 commented 9 months ago

@Utkarshn10 Are the changes correct?

Yes

0x00zer0day commented 9 months ago

Hey @0x00zer0day can you also add the screenshot of the page, rest LGTM. Thanks

Screenshot of the button right? Sure just a sec

Screenshot of the whole page.

Done

0x00zer0day commented 9 months ago

Hey @0x00zer0day i believe increasing width and height to 35px will make it look good. can you increase them. Thanks !

Funny part is, I was about to tell you the same thing.

0x00zer0day commented 9 months ago

Approved ! Thanks @0x00zer0day for contributing to Focusly :)

You're welcome! Hey just a question btw, My hacktoberfest contribution shows up as "not participating" for this PR merge, what could go wrong here?

Utkarshn10 commented 9 months ago

I have added the hacktoberfest-accepted badge in the PR. It should update now.

0x00zer0day commented 9 months ago

I have added the hacktoberfest-accepted badge in the PR. It should update now.

Oh, Thanks!

Utkarshn10 / Focusly