Skip to content
This repository has been archived by the owner on Sep 28, 2022. It is now read-only.

Fix "Fill" rounded buttons on mobile #54

Closed
shahednasser opened this issue Sep 8, 2020 · 27 comments
Closed

Fix "Fill" rounded buttons on mobile #54

shahednasser opened this issue Sep 8, 2020 · 27 comments
Labels
bug Something isn't working buttons Changes or fixes to buttons Cannot Reproduce Discussion Discuss new ideas or features good first issue Good for newcomers Hacktoberfest help wanted Extra attention is needed Priority: Medium Not too urgent, but help is needed

Comments

@shahednasser
Copy link
Collaborator

On mobile devices (specifically iPhone), when clicking on buttons of type "Fill" the overlayed color doesn't fully fill the container button.

@shahednasser shahednasser added bug Something isn't working help wanted Extra attention is needed good first issue Good for newcomers labels Sep 8, 2020
@shahednasser shahednasser pinned this issue Sep 8, 2020
@Vansh1067
Copy link

can I work on this??

@shahednasser
Copy link
Collaborator Author

Sure!

@shahednasser shahednasser added the Priority: High Items that need immediate attention label Sep 11, 2020
@shahednasser
Copy link
Collaborator Author

It seems like no one is currently working on this issue if anyone is interested

@reveurguy
Copy link

the issue you are stating doesn't seem to exist. Has it been corrected?

@shahednasser
Copy link
Collaborator Author

@reveurguy It seems that this is happening to some buttons and not others. Please look at the screenshots for examples
image
image

@shahednasser shahednasser added the buttons Changes or fixes to buttons label Sep 23, 2020
@ahmadshiddiq
Copy link

I didn't see that problem honestly, everything works well in my device.

@shahednasser
Copy link
Collaborator Author

That's interesting @ahmad-shiddiq what device are you using? Maybe this will help narrow the problem down

@shahednasser shahednasser added Discussion Discuss new ideas or features Priority: Medium Not too urgent, but help is needed Hacktoberfest and removed Priority: High Items that need immediate attention labels Sep 25, 2020
@ahmadshiddiq
Copy link

I'm sorry, I thought that was IOS. I was trying on Ipad.
WhatsApp Image 2020-09-25 at 3 07 10 PM

But I tried on my phone too, same didn't see any problem as you said.
WhatsApp Image 2020-09-26 at 4 30 28 PM

@shahednasser
Copy link
Collaborator Author

Thank you for sharing that @ahmad-shiddiq . Will keep this issue for future reference then.

@samyev
Copy link

samyev commented Oct 1, 2020

Hi, has this issue been resolved? if not, can I work on it?

@shahednasser
Copy link
Collaborator Author

@samyev it has not been resolved. We've been having trouble reproducing it. If you can reproduce and fix it then go ahead!

@kaf-lamed-beyt
Copy link
Contributor

@shahednasser what exactly is the problem?

Are the buttons too round? On mobile? Please enlighten me, so I can try fixing it

@shahednasser
Copy link
Collaborator Author

@shahednasser what exactly is the problem?

Are the buttons too round? On mobile? Please enlighten me, so I can try fixing it

The problem can be found in this comment. Some buttons are not being filled completely. This proved tricky as it could not be reproduced on all devices.

@kaf-lamed-beyt
Copy link
Contributor

@shahednasser what exactly is the problem?
Are the buttons too round? On mobile? Please enlighten me, so I can try fixing it

The problem can be found in this comment. Some buttons are not being filled completely. This proved tricky as it could not be reproduced on all devices.

Ok, I've seen it, I'd try fixing it.

I'd let you know if I can't

@ankitaugale23
Copy link

On mobile devices (specifically iPhone), when clicking on buttons of type "Fill" the overlayed color doesn't fully fill the container button.

Can I work on this?

@faraz16iqbal
Copy link

Is this issue still open?

@shahednasser
Copy link
Collaborator Author

@ankitaugale23 @faraz16iqbal you can work on it if you can reproduce it and find the solution for it (as it has been proven tricky to reproduce)

@samyev
Copy link

samyev commented Oct 2, 2020

I'm not getting it, I implemented a function for recognition of the iPhone device, but the site ends up breaking, but the problem apparently is with the iPhone, on any other device the site works very well

@shahednasser
Copy link
Collaborator Author

I'm not getting it, I implemented a function for recognition of the iPhone device, but the site ends up breaking, but the problem apparently is with the iPhone, on any other device the site works very well

Yea but someone even tried it on an iPhone and it worked fine for them, that's why it's tricky. I think this should be done with CSS but honestly I haven't figured it out either. Thank you for working on it!

@kaf-lamed-beyt
Copy link
Contributor

I'm not getting it, I implemented a function for recognition of the iPhone device, but the site ends up breaking, but the problem apparently is with the iPhone, on any other device the site works very well

You are right about this...

'cause while I was viewing it on my device (an android) it works fine

@samyev
Copy link

samyev commented Oct 2, 2020

Não estou entendendo, implementei uma função de reconhecimento do aparelho iPhone, mas o site acaba quebrando, mas o problema aparentemente é com o iPhone, em qualquer outro aparelho o site funciona muito bem

Sim, mas alguém até experimentou em um iPhone e funcionou bem para eles, é por isso que é complicado. Acho que isso deveria ser feito com CSS, mas, honestamente, também não descobri. Obrigado por trabalhar nisso!

I believe that the problem may be with the version of the IOS system then, because it runs on one and the other does not. I've been researching deeper and saw that safari on iPhone ends up breaking the CSS of the sites.

@samyev
Copy link

samyev commented Oct 2, 2020

Não estou entendendo, implementei uma função de reconhecimento do aparelho iPhone, mas o site acaba quebrando, mas o problema aparentemente é com o iPhone, em qualquer outro aparelho o site funciona muito bem

Você está certo sobre isso ...

porque enquanto eu estava visualizando no meu dispositivo (um Android), ele funciona bem

With mine too, and I realized that in some versions IOS it runs well, and in others the site does not open

@shahednasser
Copy link
Collaborator Author

Thank you for the inside @samyev . I guess we'll leave it open for now and see if we can ever find a fix for it.

@samyev
Copy link

samyev commented Oct 2, 2020

Obrigado por dentro @samyev . Acho que vamos deixar em aberto por enquanto e ver se podemos encontrar uma solução para isso.

That’s okay ><

@abhishekrawe
Copy link

yes, I m intrested on this , please assign me.

@basvandenwijngaard
Copy link

basvandenwijngaard commented Oct 4, 2020

I'm able to reproduce this with rounded buttons with the fill-animation effect (https://sbuttons.github.io/sbuttons/#fill-color-left) on an iPhone Xs, but also on Safari on macOs. I also have a possible fix for it.

@shahednasser
Copy link
Collaborator Author

Issue finally resolved. Thank you everyone!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working buttons Changes or fixes to buttons Cannot Reproduce Discussion Discuss new ideas or features good first issue Good for newcomers Hacktoberfest help wanted Extra attention is needed Priority: Medium Not too urgent, but help is needed
Projects
None yet
Development

No branches or pull requests

10 participants