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

Add nub to chat bubbles #2137

Merged
merged 21 commits into from
Jul 18, 2019
Merged

Add nub to chat bubbles #2137

merged 21 commits into from
Jul 18, 2019

Conversation

compulim
Copy link
Contributor

@compulim compulim commented Jul 2, 2019

Fixes #1658.

Changelog Entry

Description

Bubble nub is a popular ask from the community. We have added a nub which can be customized by its size, alignment, and offset to the corner.

  • New styleOptions introduced
    • bubbleNubSize and bubbleFromUserNubSize is a number in pixels
    • bubbleNubOffset and bubbleFromUserNubOffset can be either
      • "bottom" or -0
      • "top" or 0
      • A positive number means offset from top
      • A negative number means offset from number
    • bubbleFromUserNubSize, bubbleFromUserNubOffset
  • styleOptions renamed or modified
    • bubbleBorder now becomes bubbleBorderColor, bubbleBorderStyle, and bubbleBorderWidth
      • Same for bubbleFromUserBorder
      • Instead of able to style 4 borders individually, you can only style all 4 borders at once from now on

image

image

image

Specific Changes


  • Testing Added

@compulim compulim added the 4.6 label Jul 2, 2019
@corinagum corinagum added this to Unassigned in R8 Prioritization Jul 10, 2019
@compulim compulim marked this pull request as ready for review July 10, 2019 22:05
@corinagum
Copy link
Contributor

I think we're going to need documentation on this either in defaultStyleOptions or in our branding styling sample.

@corinagum corinagum moved this from Unassigned to In progress in R8 Prioritization Jul 11, 2019
@compulim compulim self-assigned this Jul 12, 2019
@coveralls
Copy link

coveralls commented Jul 12, 2019

Coverage Status

Coverage increased (+1.2%) to 64.541% when pulling 484c677 on compulim:feat-bubble-nub into 84d194b on microsoft:master.

@corinagum
Copy link
Contributor

image
Is this type of nub supported?

@compulim
Copy link
Contributor Author

image
Is this type of nub supported?

This one is not supported. Would need a totally different way to layout, but it seems fun extension to this PR. Thanks for sharing.

Copy link
Contributor

@corinagum corinagum left a comment

Choose a reason for hiding this comment

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

I'm not a huge fan of the styling of the nub. :/ Did you do a design pass for this? Maybe discuss with @DesignPolice to see what changes might happen with the Fluent update?

samples/01.a.getting-started-full-bundle/index.html Outdated Show resolved Hide resolved
packages/playground/src/App.js Outdated Show resolved Hide resolved
packages/component/src/Activity/CarouselFilmStrip.js Outdated Show resolved Hide resolved
@compulim
Copy link
Contributor Author

Talked to @DesignPolice before about the nub design and he is good with the current one.

@alokraj68
Copy link

This should be having a middle option like the top and bottom right? In V3 it was in the middle of the blob.

@arman-g
Copy link

arman-g commented Dec 4, 2019

Hi, is this spouse to work with CDN? I am using CDN v4.6.0 and these options do not work:

bubbleNubOffset: 'bottom',
bubbleNubSize: 10,

but these do work:

bubbleFromUserNubOffset: 'bottom',
bubbleFromUserNubSize: 10,

What am I doing wrong or is it a bug?
nub

@corinagum
Copy link
Contributor

You'll have to also add showNub: true to your style options

@arman-g
Copy link

arman-g commented Dec 4, 2019

@corinagum I did add showNub: true but it did not help.

nub

@corinagum
Copy link
Contributor

@arman-g thanks. Odd that it's working for user but not bot. Could you file an issue so I can add it to the investigate list? We have several issues up in the air right now but I'll get to this asap.

@arman-g
Copy link

arman-g commented Dec 4, 2019

@corinagum just did issue #2667

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Add message bubble arrows CSS
5 participants