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

Merge the two different versions of _view_lineup.scss #403

Merged
merged 3 commits into from
Jul 29, 2020

Conversation

oltionchampari
Copy link
Contributor

@oltionchampari oltionchampari commented Jul 29, 2020

Apparently the file _view_lineup.scss exists with two different histories. The current develop branch is missing the changes made to the file the past year, that are present in the file with alternative history.

develop branch: https://github.com/datavisyn/tdp_core/commits/develop/src/styles/_view_lineup.scss
alternative history: https://github.com/datavisyn/tdp_core/commits/b9d40d04c23ef53fd2693ec8d3f93da1068a7f2d/src/styles/_view_lineup.scss
last common commit is 768cf0c from Aug 28, 2019.

This PR adds the changes missing from the current _view_lineup.scss of the develop branch.
Tested branch in ordino and everything looks as it should

thinkh and others added 2 commits July 29, 2020 11:20
There is an alternative version of this file in the develop branch (with commit b9d40d0) that contains already some fixes. We copied it here now.

We do not understand yet how we got an alternative history for this file in the first place.
@oltionchampari oltionchampari added the type: bug Something isn't working label Jul 29, 2020
line-height: 1;
}

// Select only first div containing the `lineup-engine`.
// Do not select the `div.lu-backdrop` element,
// which is moved one level up in the DOM tree in ARankingView.ts
> div:first-of-type {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Took the most recent solution from @puehringer

Copy link
Member

@thinkh thinkh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works. 👍

@thinkh thinkh merged commit b26d93a into develop Jul 29, 2020
@thinkh thinkh deleted the thinkh/fix-lineup-scss branch July 29, 2020 14:15
@thinkh thinkh mentioned this pull request Jul 30, 2020
48 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants