Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

fix: word wrap in SendActivity#2908

Merged
cwhitten merged 4 commits intomasterfrom
visual/text-wrap
May 6, 2020
Merged

fix: word wrap in SendActivity#2908
cwhitten merged 4 commits intomasterfrom
visual/text-wrap

Conversation

@yeze322
Copy link
Contributor

@yeze322 yeze322 commented May 6, 2020

Description

closes #2875

Task Item

image

image

Screenshots

@yeze322 yeze322 changed the base branch from stable to master May 6, 2020 05:24
@github-actions
Copy link

github-actions bot commented May 6, 2020

Coverage Status

Coverage remained the same at 0.0% when pulling 7f5a9db on visual/text-wrap into 70b9715 on master.

@yeze322 yeze322 linked an issue May 6, 2020 that may be closed by this pull request
@a-b-r-o-w-n a-b-r-o-w-n self-assigned this May 6, 2020
a-b-r-o-w-n
a-b-r-o-w-n previously approved these changes May 6, 2020
@a-b-r-o-w-n a-b-r-o-w-n added the Approved to merge approved, waiting to be merged label May 6, 2020
white-space: initial;
overflow-wrap: break-word;
word-break: break-all;
word-break: break-word;
Copy link
Contributor

Choose a reason for hiding this comment

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

MDN says break-word is deprecated and to use word-break: normal with overflow-wrap: anywhere instead.

Copy link
Contributor

Choose a reason for hiding this comment

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

@beyackle would you mind pushing a fix for that and verifying the behavior is the same? I'd like to get this merged today.

Copy link
Contributor

Choose a reason for hiding this comment

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

image

Works just fine (text from an MDN page I had open at the time). I'll push this up here.

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks @beyackle

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks @beyackle !

@a-b-r-o-w-n a-b-r-o-w-n removed the Approved to merge approved, waiting to be merged label May 6, 2020
@cwhitten cwhitten merged commit 52a54a4 into master May 6, 2020
@cwhitten cwhitten deleted the visual/text-wrap branch May 6, 2020 17:09
alanlong9278 added a commit that referenced this pull request May 7, 2020
* master: (58 commits)
  fix: Copy skill manifests to the correct directory in the localPublish plugin (#2932)
  feat: Goto Begin Dialog after clicking dialog (#2922)
  fix: Improved Electron auto update UX (#2925)
  fix: Action Flow gradual left alignment (#2909)
  fix: word wrap in SendActivity (#2908)
  fix: Fixed various onboarding issues and updated content (#2900)
  chore: Component Governance (#2899)
  perf: improve property editor performance (#2921)
  fix: paste blank node (#2905)
  extract memory variables at lg lsp server (#2902)
  feat: manage samples via plugin (#2805)
  can not use event capture in visual editor (#2913)
  style: make focus styles more consistent (#2898)
  feat: azure publish plugin (#2733)
  fix: unable to clear form title (#2885)
  fix: Populate env variable with AppData folder (#2894)
  a11y: use Key/Value aria labels in object field (#2890)
  Fix border issue in visual editor (#2891)
  fix: changes manifest type from '.manifest' to '.json' (#2888)
  Fixed packaged folder structure. (#2887)
  ...
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
* update word-break css

* use non-deprecated CSS

Co-authored-by: Andy Brown <asbrown002@gmail.com>
Co-authored-by: Ben Yackley <61990921+beyackle@users.noreply.github.com>
Co-authored-by: Chris Whitten <christopher.whitten@microsoft.com>
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.

Cannot use multi-selection when Visual Editor container is focused Improve SendActivity word wrap

4 participants