Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove margin:auto #788

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

leigh123linux
Copy link
Contributor

This has been removed in cinnamon-3.4

linuxmint/cinnamon@b579bbe

(cinnamon:1567): St-WARNING **: Ignoring length property that isn't a number at line 967, col 11

fossfreedom referenced this pull request in arc-design/arc-theme Nov 18, 2017
Upstream PR #788 - This avoids warning messages as margin:auto
is not supported in cinnamon 3.4 and later
@fossfreedom
Copy link

Many thanks @leigh123linux - this has been merged in our soft fork https://github.com/NicoHood/arc-theme . If you can, please help out maintaining the cinnamon section of the theme. cheers

@NicoHood
Copy link

@fossfreedom Can you please merge those changes via PRs on the fork? this way we can track them better. My wish would be that I check your PRs and you mine. This way we know what each other did and review/quick check it also. At least a separate PR would make sense to me.

@fossfreedom
Copy link

Sure. Will do a hard reset and convert to PRs

fossfreedom referenced this pull request in arc-design/arc-theme Nov 19, 2017
Upstream PR #788 - This avoids warning messages as margin:auto
is not supported in cinnamon 3.4 and later
@NicoHood
Copy link

@leigh123linux Thanks! You can close this issue now, as it got merged in the new upstream:
https://github.com/NicoHood/arc-theme

jnsh pushed a commit to jnsh/arc-theme that referenced this pull request Mar 25, 2019
Upstream PR horst3180#788 - This avoids warning messages as margin:auto
is not supported in cinnamon 3.4 and later
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants