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

Button group alignment doesn't work as expected #401

Closed
mghenciu opened this issue Aug 5, 2020 · 13 comments
Closed

Button group alignment doesn't work as expected #401

mghenciu opened this issue Aug 5, 2020 · 13 comments
Assignees
Labels
bug Something isn't working

Comments

@mghenciu
Copy link

mghenciu commented Aug 5, 2020

Repro:

  1. Update Otter to version 1.5.6
  2. Go to this demo https://demosites.io/laundry-services-gb/
  3. Notice the Button group alignment, from the Header section on the Homepage

Expected
image

In version 1.5.6
Screenshot 2020-08-05 at 16 25 28

@rodica-andronache
Copy link

might be related to Codeinwp/neve#1870. For me, after going into the editor and save the page, everything is ok

@mghenciu
Copy link
Author

mghenciu commented Aug 6, 2020

Thanks Rodica!

Yes, going into the editor and saving the page + Regenerate CSS from Otter options, fixes this problem.

@HardeepAsrani
Copy link
Member

Should I close it then @mghenciu?

@selul selul reopened this Aug 18, 2020
@selul selul added the bug Something isn't working label Aug 18, 2020
@selul
Copy link
Contributor

selul commented Aug 18, 2020

this is still present and the regenerate issue is still present.

You can replicate this by building some button group block on the old version of otter and update to the new one after. The style for the button group is lost or somehow not applied.

It seems that it heals after you manually update the post, by going to editor and click update, BUT NOT when we regenerate the CSS style.

It seems that the action of updating inside the editor alters the post_content and fix the issues, action that is not done using the regenerate CSS since this is not being done in the backend.

I have attached the post_content before clicking the update and after and if you run a diff on it you can see that some ids are changed.
Archive.zip

@HardeepAsrani how we can fix this? It isn't a solution to ask people to go to their pages and click update.

@mghenciu
Copy link
Author

After I started manually Re-saving the pages, the alignment for Button-groups it's fixed.
But there is another challenge with border-radius for button-groups. Basically buttons have a 3px border radius, even if in fron-end it looks ok (set to 0) and in customizer the border radius is set to 0.
The border-radius looks to be coming from here: https://s33834.pcdn.co/fitness-trainer-gb/wp-content/themes/neve/style.min.css?ver=2.7.7
@abaicus Maybe you have an idea if this comes from Neve or from Otter.

@selul
The question now it's should I continue Re-saving pages (to fix the alignment issue) and leave the border-radius for later, or?

Customizer
image (5)

Front-end
image (4)

Live site:
image (3)

How it should look:
image

@mghenciu
Copy link
Author

mghenciu commented Aug 27, 2020

Another thing that's worth mentioning here (regarding the button-group style reset), is that there is a pattern, usually the Reusable Blocks have the style broken - even after you resave the page, but the other ones (not part of a Reusable Block) are fixed after resaving the page.

How it looks:
image

How it should look:
image

@selul
Copy link
Contributor

selul commented Aug 27, 2020

@mghenciu how about if you click edit on the reusable block and save them?

@mghenciu
Copy link
Author

Yep, @selul . Already doing this where we have Reusable blocks (and it works) and I am re-adding the buttons/sections were things don't get fixed after resave.
Just wanted to bring this up so everyone is aware. I am not sure if this affects users in any way.

@HardeepAsrani
Copy link
Member

@mghenciu Did you click to Edit the Reusable blocks and then Save? Not the post but the Reusable Block. :)

@mghenciu
Copy link
Author

@HardeepAsrani Yes, Edit the Reusable blocks and then Save.
And for the pages with no Reusable blocks, Re-saving the post/page fixed the problem (in most cases, because there were some instances where the buttons were empty, so they needed to be rebuild from 0).
And the border-radius issue is still there.

@HardeepAsrani
Copy link
Member

@mghenciu That is fixed in development and will be fixed in the upcoming release. Hopefully this week.

@mghenciu
Copy link
Author

@HardeepAsrani , the issue with the buttons aligning to the left after import, 'made his presence' again on 3 demos we tested so far:
https://musical-bobolink.jurassic.ninja/ >>> original demo: https://demosites.io/photography-gb
https://abundant-tarantula.jurassic.ninja/ >>> original demo: https://demosites.io/rokophoto-gb/
https://demosites.io/laundry-services-gb/

Basically after importing the demos, some buttons are aligned to the left instead of center.

Rokophoto:
Screenshot 2021-03-12 at 09 39 19
Screenshot 2021-03-12 at 09 39 07
Screenshot 2021-03-12 at 09 38 42

Laundry Services: https://prnt.sc/10jjrhh

And yes, I resaved the pages but the issue is still present. Will try to regenerate, and see if this fixes the issue.
My question now is if I just should fix this demos (by manually re-building those sections ) or you can look more into it, maybe it's a general issue with all Gb demos? maybe it's worth a Mass 'css regeneration'

cc @tudor-themeisle

@HardeepAsrani
Copy link
Member

Issue was made here regarding this: #614

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants