Skip to content

Details List - Column Drag drop changes#5502

Merged
ThomasMichon merged 25 commits intomicrosoft:5.0from
laxmikankanala:laxmi/backport
Jul 11, 2018
Merged

Details List - Column Drag drop changes#5502
ThomasMichon merged 25 commits intomicrosoft:5.0from
laxmikankanala:laxmi/backport

Conversation

@laxmikankanala
Copy link
Copy Markdown
Contributor

@laxmikankanala laxmikankanala commented Jul 10, 2018

Porting 3 PRs from version 6 to support column drag and drop in Details List

List of PRs:
#4857
#5309
#5432

Microsoft Reviewers: Open in CodeFlow

@cliffkoh
Copy link
Copy Markdown
Contributor

Please feel free to merge. You can ignore the Appveyor failure.

Copy link
Copy Markdown
Contributor

@kenotron kenotron left a comment

Choose a reason for hiding this comment

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

Can you please wait until my change is thru before merging this? I've had to redo the styling of DL Header to CSS-in-JS. This will affect styles I had to port by hand a bit it seems (I'm hitting "request changes" until I get mine in)

Copy link
Copy Markdown
Contributor

@kenotron kenotron left a comment

Choose a reason for hiding this comment

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

Just realized this is targeting 5.0, rescinding my block

@kenotron kenotron dismissed their stale review July 10, 2018 22:16

it's for 5.0

@ThomasMichon ThomasMichon merged commit d914311 into microsoft:5.0 Jul 11, 2018
@microsoft microsoft locked as resolved and limited conversation to collaborators Aug 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants