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

Fixing summary ajax row animation direction #144

Merged
merged 1 commit into from
Jun 28, 2017
Merged

Conversation

davidkpiano
Copy link
Contributor

Continuation of #121

jun-28-2017 15-26-49

@davidkpiano davidkpiano added this to the M022 milestone Jun 28, 2017
@davidkpiano davidkpiano self-assigned this Jun 28, 2017
@davidkpiano davidkpiano requested a review from legomushroom June 28, 2017 19:28
@msftclas
Copy link

@davidkpiano,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

}
.glimpse-section-summary {
.glimpse-ajax-rows {
margin-left: 20px !important;
width: 180px !important;
flex-grow: 1 !important;
table-layout: fixed !important;
Copy link
Member

Choose a reason for hiding this comment

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

Do we need this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, otherwise the table doesn't obey 180px. I'd like to fix the width so it stops jumping around.

Copy link
Member

@legomushroom legomushroom left a comment

Choose a reason for hiding this comment

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

One question, LGTM.

@davidkpiano davidkpiano merged commit 3e557b1 into dev Jun 28, 2017
@avanderhoorn avanderhoorn deleted the davidkpiano-121-2 branch July 14, 2017 21:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants