Skip to content

Conversation

@VincentBailly
Copy link
Member

Description of changes

The persona control was bigger than the personaCoin in order to include the border of the presence indicator. However the presence indicator has an absolute position so the layout should behave as if it does not exist.

@mtennoe
Copy link
Member

mtennoe commented Feb 1, 2018

@dzearing, @betrue-final-final : Seems like screener is not able to highlight the differences. Is there a known screener issue? Its very hard to see the diff at all here

@lynamemi lynamemi closed this Feb 5, 2018
@lynamemi lynamemi reopened this Feb 5, 2018
@lynamemi
Copy link
Collaborator

lynamemi commented Feb 5, 2018

Closing and reopening retriggers the build and might get you through a clean screener check. When this PR first went in, I did see a diff that I am not seeing anymore. It could have been coming from master if it passes this time.

**/
max-width: 100%;
overflow-x: hidden;
overflow: hidden;
Copy link
Collaborator

Choose a reason for hiding this comment

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

looks like this line is what's blocking in screener. Is there another way to accomplish what you need here?

@dzearing dzearing closed this Feb 6, 2018
@dzearing dzearing reopened this Feb 6, 2018
@VincentBailly VincentBailly reopened this Feb 6, 2018
@mtennoe mtennoe closed this Feb 7, 2018
@mtennoe mtennoe reopened this Feb 7, 2018
@VincentBailly VincentBailly reopened this Feb 7, 2018
@dzearing dzearing merged commit 2514f15 into microsoft:master Feb 12, 2018
@microsoft microsoft locked as resolved and limited conversation to collaborators Aug 31, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants