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

BUG: Border at top of tabs when no subtabs #925

Merged
merged 1 commit into from Nov 4, 2012

Conversation

adrexia
Copy link
Contributor

@adrexia adrexia commented Nov 4, 2012

This bug reappeared when branches were merged because the 3.0 version of _style.scss had tabsets defined in different places.

This scss was inserted, but there was no idication that it was new to master (except in the resulting css file). As these styles are declared differently elsewhere in _style.scss, I think this declaration must have been refactored in an earlier commit in master.

This bug reappeared when branches were merged because the 3.0 version
of _style.scss had tabsets defined in different places.

This scss was inserted, but there was no idication that it was new to
master (except in the resulting css file). As these styles are declared
differently elsewhere in _style.scss, I think this declaration must
have been refactored in an earlier commit in master.
@chillu
Copy link
Member

chillu commented Nov 4, 2012

Probably my fault, its hard to merge SCSS without actually tracking down the original commits changing it. In other news, I've consolidated the dozen tab SCSS definitions and overrides a bit during the hackfest, should be able to send a pull request tomorrow.

chillu added a commit that referenced this pull request Nov 4, 2012
BUG: Border at top of tabs when no subtabs
@chillu chillu merged commit 6bb7a95 into silverstripe:master Nov 4, 2012
@adrexia
Copy link
Contributor Author

adrexia commented Nov 4, 2012

Understand. The very idea of having to merge two diverged branches of SCSS/CSS based changes gives me nightmares. ;)

That'll be nice to see!

@adrexia
Copy link
Contributor Author

adrexia commented Nov 5, 2012

Arg. Deleted too much of that tab styling. Was only meaning to remove the bottom part. New commit coming.

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.

None yet

2 participants