fix: ThemingV2 beta adjustments #1728
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov Report
@@ Coverage Diff @@
## theming-v2-beta #1728 +/- ##
================================================
Coverage 83.60% 83.60%
================================================
Files 253 253
Lines 6220 6220
Branches 1670 1670
================================================
Hits 5200 5200
Misses 864 864
Partials 156 156
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
arnautov-anton
added a commit
that referenced
this pull request
Aug 30, 2022
* fix: adjust variable names * fix(PopperTooltip): add type export, document properties, remove unused className
github-actions bot
pushed a commit
that referenced
this pull request
Sep 6, 2022
# [10.0.0-theming-v2.3](v10.0.0-theming-v2.2...v10.0.0-theming-v2.3) (2022-09-06) ### Bug Fixes * include mdast-util-find-and-replace into our CJS bundle ([#1702](#1702)) ([#1703](#1703)) ([8010889](8010889)), closes [#1698](#1698) * prevent double submissions in korean ([#1720](#1720)) ([5d781d8](5d781d8)) * ThemingV2 beta adjustments ([#1728](#1728)) ([785ee11](785ee11)) * **ThemingV2:** MessageInputFlat missing lodash/zipObject ([#1721](#1721)) ([dd8a457](dd8a457)) * **Vite:** add emoji-mart (emoji, picker) re-export ([#1724](#1724)) ([c90cf4b](c90cf4b)) ### Features * increase and support overriding jump to message limit ([#1718](#1718)) ([8c720f4](8c720f4)) * **ThemingV2:** PopperTooltip component ([#1714](#1714)) ([9b6301e](9b6301e))
Merged
github-actions bot
pushed a commit
that referenced
this pull request
Sep 9, 2022
# [10.0.0](v9.5.1...v10.0.0) (2022-09-09) ### Bug Fixes * add missing class str-chat__message-actions-list-item-button to CustomMessageActions root ([be16e40](be16e40)) * add missing getChannels callback dependency "searching" ([a3307f7](a3307f7)) * add str-chat__message--other class to MessageDeleted ([25f3190](25f3190)) * **Card:** prefer title_link over og_scrape_url ([843990e](843990e)) * do not generate class names to contain string 'undefined', do not pass Media prop to Card ([40342fe](40342fe)) * **EditMessageForm:** remove circular dependency ([6218a65](6218a65)) * exit with non-zero status code if failed to provide args to merge-stream-chat-css-docs.sh ([a4719b9](a4719b9)) * File attachment UI in theme-v1 ([0a80bef](0a80bef)) * File attachment UI in theme-v1 ([9604ca6](9604ca6)) * **FilePreviewItem:** add file type for correct file icons ([6e6fce5](6e6fce5)) * forward SearchInput prop to SearchBar ([16fc8f8](16fc8f8)) * improve scrollToBottom with image attachments ([be8bb7a](be8bb7a)) * **MessageInput:** add container className ([a5e7908](a5e7908)) * **MessageInputFlat:** send button adjustments ([f456704](f456704)) * **MessageInput:** remove useId, add quotedMessage patch ([64e07d3](64e07d3)) * **MessageInput:** update dropzone markup ([974802b](974802b)) * **MessageList:** prevent redundant calls to scroll to bottom, don't use ResizeObserver ([363676e](363676e)) * **MessageStatus:** add V2 TooltipContainer component "shim" ([dcfbbfb](dcfbbfb)) * move card caption to card content and rename to source link ([c44bcd7](c44bcd7)) * **QuotedMessagePreview:** use themingVersion instead of PreviewHeader property ([4f79b07](4f79b07)) * reduce mount/unmount of image attachments ([34082a4](34082a4)) * Responsive layout ([7551650](7551650)) * show channel list if search input contains empty string ([b73dd2c](b73dd2c)) * **SuggestionList:** update trigger limits ([2e1f025](2e1f025)) * sync event listener keyDown type btw the image attachment and gallery modal ([51e7c14](51e7c14)) * ThemingV2 beta adjustments ([#1728](#1728)) ([785ee11](785ee11)) * **ThemingV2:** MessageInputFlat missing lodash/zipObject ([#1721](#1721)) ([dd8a457](dd8a457)) * **TypingIndicator:** adjust position of the indicator ([f5db199](f5db199)) * **TypingIndicator:** use MessageListMainPanel to position the indicator ([865cbc8](865cbc8)) * Merge pull request #1697 from GetStream/theming-v2-user-testing ([2c133ad](2c133ad)), closes [#1697](#1697) * Remove useMobilePress and useBreakpoint hooks (#1648) ([430bf24](430bf24)), closes [#1648](#1648) ### Features * adapt MessageOptions to theming v2 ([23c2d93](23c2d93)) * adapt MessageStatus to theming v2 ([a5b0fae](a5b0fae)) * add "str-chat__message--error-message" class to message error div ([c4f7520](c4f7520)) * add Attachment icons for theming v2 ([97e8047](97e8047)) * add Card component for theming v2 ([5f5341f](5f5341f)) * add class "str-chat__message-sender-avatar" to Avatar root to display it for sender only ([ecd0b4b](ecd0b4b)) * add class str-chat__message-list-scroll to virtuoso root element ([7d2284e](7d2284e)) * add common IconProps type ([3ff89bb](3ff89bb)) * add FileAttachment component for theming v2 ([52acd80](52acd80)) * add group styles to virtualized message list items ([ff2044e](ff2044e)) * add Message icons for theming v2 - MessageDeliveredIcon, MessageErrorIcon ([9ff9034](9ff9034)) * add ModalGallery to the ComponentContext ([27e149a](27e149a)) * add realistic giphy attachment generator ([e4c2a7b](e4c2a7b)) * add str-chat__simple-message--error-failed class to str-chat__message-inner ([a5f8f94](a5f8f94)) * add str-chat-react__modal__inner class to str-chat__modal__inner ([0234522](0234522)) * add svg image class to attachment ([3d0237d](3d0237d)) * add theme v2 class to CustomNotification ([037dc89](037dc89)) * add theme v2 to Reaction components, extract shared logic ReactionList & SimpleReactionList ([f6a12d0](f6a12d0)) * add themeVersion flag to ChatProps & ChatContext ([70cbfcb](70cbfcb)) * add ThemeVersion type to ChatContext ([695f30a](695f30a)) * add theming v2 changes for channel and channel header ([#1632](#1632)) ([3f8fddb](3f8fddb)) * add theming v2 classes to MessageActions elements ([d62e04a](d62e04a)) * add theming v2 classes to QuotedMessage ([8c5d2ff](8c5d2ff)) * add theming v2 classes to ReactionSelector & ReactionList ([fbedb42](fbedb42)) * add v2 classes to send and cancel button of EditMessageForm ([#1669](#1669)) ([ab75c2c](ab75c2c)) * adjust Audio widget for theming v2 ([f08c6f5](f08c6f5)) * adjust Gallery and Image widget for theming v2 ([de29a73](de29a73)) * adjust MessageRepliesCountButton to theming v2, add classes ([5076fd5](5076fd5)) * adjust MessageSimple for theming v2 ([ebd4bd7](ebd4bd7)) * allow card image enlargement in modal ([47bf301](47bf301)) * apply theme-v2 to channel list and preview ([#1603](#1603)) ([cc88f1f](cc88f1f)) * change the close icon for modal and remove Close text ([88a5f7c](88a5f7c)) * compute the themeVersion value, remove themeVersion Chat prop ([3421087](3421087)) * convert attachment render functions into components, group attachments in order ([aeee078](aeee078)) * do not sanitize attachment scrape urls ([aa1624a](aa1624a)) * extract CardAudio and render only uploaded audio data in Audio component ([8027908](8027908)) * forward flag "disabled" to search input ([2ec25a1](2ec25a1)) * include the parent message in virtualized scrollable message list ([dd63427](dd63427)) * message is considered top if it has reactions and bottom if the next message has reactions ([638aead](638aead)) * **MessageInput:** add drag & drop upload functionality ([e731b67](e731b67)) * **ProgressBar:** add "seeking" feature to the progress bar ([0320864](0320864)) * **ProgressBar:** add onClick property ([4d9d06c](4d9d06c)) * remove avatar from the thread header ([dec0d8d](dec0d8d)) * remove deprecated components: MessageCommerce, MessageLivestream, MessageTeam ([9d75fb8](9d75fb8)) * remove translations for deprecated components: MessageCommerce, MessageLivestream, MessageTeam ([e524d0a](e524d0a)) * render cards for each attachment with scraped data ([0a59806](0a59806)) * show always ScrollToBottomButton on scroll up and show unread message count ([e554356](e554356)) * stop using FixedHeightMessage as default VirtualMessage component ([fc67915](fc67915)) * switch ladle to v2 ([ecd1cc6](ecd1cc6)) * **theming-v2:** add channel search for theme v2 ([#1685](#1685)) ([b735c30](b735c30)), closes [#1669](#1669) * **ThemingV2:** PopperTooltip component ([#1714](#1714)) ([9b6301e](9b6301e)) * **TypingIndicator:** add translations ([f079e26](f079e26)) * update @stream-io/stream-chat-css to v3.0.0 ([c5e392c](c5e392c)) * update message componets with theme v2 designs ([e5192d5](e5192d5)) * use FileIcon with version in UploadsPreview ([4d150b1](4d150b1)) * wrap ThreadHead content in a div to enable styling for class str-chat__parent-message-li ([9323edb](9323edb)) ### Reverts * fix: File attachment UI in theme-v1 ([789dd27](789dd27)) ### BREAKING CHANGES * ThemingV2 - user testing and adjustments * useMobilePress and useBreakpoint hooks are removed. useMobilePress: Historically, this hook programmatically handled the user interaction with Message components by toggling `mobile-press` class upon user interaction. The goal of this operation was to have the message actions displayed on the screen. Internally, we found a better solution by offloading this behavior to the browser and utilizing `:focus` and `:focus-within` CSS pseudo-selectors. useBreakpoint: This hook did hold the "programmatic" responsive UI breakpoints. We realized they aren't always in line with our stylesheet breakpoints and possibly with our customer's breakpoints. This misalignment was causing some inconsistencies and issues. We are removing this hook because we believe defining UI breakpoints should be responsibility of our customers. SearchResults: During the refactoring, we stumbled upon one side-effect where `popupResults` prop wasn't always respected. The fix of it could be a breaking change for a small percentage of our customers, but we believe this fix is the right thing to do.
arnautov-anton
pushed a commit
that referenced
this pull request
Sep 12, 2022
# [10.0.0](v9.5.1...v10.0.0) (2022-09-09) ### Bug Fixes * add missing class str-chat__message-actions-list-item-button to CustomMessageActions root ([be16e40](be16e40)) * add missing getChannels callback dependency "searching" ([a3307f7](a3307f7)) * add str-chat__message--other class to MessageDeleted ([25f3190](25f3190)) * **Card:** prefer title_link over og_scrape_url ([843990e](843990e)) * do not generate class names to contain string 'undefined', do not pass Media prop to Card ([40342fe](40342fe)) * **EditMessageForm:** remove circular dependency ([6218a65](6218a65)) * exit with non-zero status code if failed to provide args to merge-stream-chat-css-docs.sh ([a4719b9](a4719b9)) * File attachment UI in theme-v1 ([0a80bef](0a80bef)) * File attachment UI in theme-v1 ([9604ca6](9604ca6)) * **FilePreviewItem:** add file type for correct file icons ([6e6fce5](6e6fce5)) * forward SearchInput prop to SearchBar ([16fc8f8](16fc8f8)) * improve scrollToBottom with image attachments ([be8bb7a](be8bb7a)) * **MessageInput:** add container className ([a5e7908](a5e7908)) * **MessageInputFlat:** send button adjustments ([f456704](f456704)) * **MessageInput:** remove useId, add quotedMessage patch ([64e07d3](64e07d3)) * **MessageInput:** update dropzone markup ([974802b](974802b)) * **MessageList:** prevent redundant calls to scroll to bottom, don't use ResizeObserver ([363676e](363676e)) * **MessageStatus:** add V2 TooltipContainer component "shim" ([dcfbbfb](dcfbbfb)) * move card caption to card content and rename to source link ([c44bcd7](c44bcd7)) * **QuotedMessagePreview:** use themingVersion instead of PreviewHeader property ([4f79b07](4f79b07)) * reduce mount/unmount of image attachments ([34082a4](34082a4)) * Responsive layout ([7551650](7551650)) * show channel list if search input contains empty string ([b73dd2c](b73dd2c)) * **SuggestionList:** update trigger limits ([2e1f025](2e1f025)) * sync event listener keyDown type btw the image attachment and gallery modal ([51e7c14](51e7c14)) * ThemingV2 beta adjustments ([#1728](#1728)) ([785ee11](785ee11)) * **ThemingV2:** MessageInputFlat missing lodash/zipObject ([#1721](#1721)) ([dd8a457](dd8a457)) * **TypingIndicator:** adjust position of the indicator ([f5db199](f5db199)) * **TypingIndicator:** use MessageListMainPanel to position the indicator ([865cbc8](865cbc8)) * Merge pull request #1697 from GetStream/theming-v2-user-testing ([2c133ad](2c133ad)), closes [#1697](#1697) * Remove useMobilePress and useBreakpoint hooks (#1648) ([430bf24](430bf24)), closes [#1648](#1648) ### Features * adapt MessageOptions to theming v2 ([23c2d93](23c2d93)) * adapt MessageStatus to theming v2 ([a5b0fae](a5b0fae)) * add "str-chat__message--error-message" class to message error div ([c4f7520](c4f7520)) * add Attachment icons for theming v2 ([97e8047](97e8047)) * add Card component for theming v2 ([5f5341f](5f5341f)) * add class "str-chat__message-sender-avatar" to Avatar root to display it for sender only ([ecd0b4b](ecd0b4b)) * add class str-chat__message-list-scroll to virtuoso root element ([7d2284e](7d2284e)) * add common IconProps type ([3ff89bb](3ff89bb)) * add FileAttachment component for theming v2 ([52acd80](52acd80)) * add group styles to virtualized message list items ([ff2044e](ff2044e)) * add Message icons for theming v2 - MessageDeliveredIcon, MessageErrorIcon ([9ff9034](9ff9034)) * add ModalGallery to the ComponentContext ([27e149a](27e149a)) * add realistic giphy attachment generator ([e4c2a7b](e4c2a7b)) * add str-chat__simple-message--error-failed class to str-chat__message-inner ([a5f8f94](a5f8f94)) * add str-chat-react__modal__inner class to str-chat__modal__inner ([0234522](0234522)) * add svg image class to attachment ([3d0237d](3d0237d)) * add theme v2 class to CustomNotification ([037dc89](037dc89)) * add theme v2 to Reaction components, extract shared logic ReactionList & SimpleReactionList ([f6a12d0](f6a12d0)) * add themeVersion flag to ChatProps & ChatContext ([70cbfcb](70cbfcb)) * add ThemeVersion type to ChatContext ([695f30a](695f30a)) * add theming v2 changes for channel and channel header ([#1632](#1632)) ([3f8fddb](3f8fddb)) * add theming v2 classes to MessageActions elements ([d62e04a](d62e04a)) * add theming v2 classes to QuotedMessage ([8c5d2ff](8c5d2ff)) * add theming v2 classes to ReactionSelector & ReactionList ([fbedb42](fbedb42)) * add v2 classes to send and cancel button of EditMessageForm ([#1669](#1669)) ([ab75c2c](ab75c2c)) * adjust Audio widget for theming v2 ([f08c6f5](f08c6f5)) * adjust Gallery and Image widget for theming v2 ([de29a73](de29a73)) * adjust MessageRepliesCountButton to theming v2, add classes ([5076fd5](5076fd5)) * adjust MessageSimple for theming v2 ([ebd4bd7](ebd4bd7)) * allow card image enlargement in modal ([47bf301](47bf301)) * apply theme-v2 to channel list and preview ([#1603](#1603)) ([cc88f1f](cc88f1f)) * change the close icon for modal and remove Close text ([88a5f7c](88a5f7c)) * compute the themeVersion value, remove themeVersion Chat prop ([3421087](3421087)) * convert attachment render functions into components, group attachments in order ([aeee078](aeee078)) * do not sanitize attachment scrape urls ([aa1624a](aa1624a)) * extract CardAudio and render only uploaded audio data in Audio component ([8027908](8027908)) * forward flag "disabled" to search input ([2ec25a1](2ec25a1)) * include the parent message in virtualized scrollable message list ([dd63427](dd63427)) * message is considered top if it has reactions and bottom if the next message has reactions ([638aead](638aead)) * **MessageInput:** add drag & drop upload functionality ([e731b67](e731b67)) * **ProgressBar:** add "seeking" feature to the progress bar ([0320864](0320864)) * **ProgressBar:** add onClick property ([4d9d06c](4d9d06c)) * remove avatar from the thread header ([dec0d8d](dec0d8d)) * remove deprecated components: MessageCommerce, MessageLivestream, MessageTeam ([9d75fb8](9d75fb8)) * remove translations for deprecated components: MessageCommerce, MessageLivestream, MessageTeam ([e524d0a](e524d0a)) * render cards for each attachment with scraped data ([0a59806](0a59806)) * show always ScrollToBottomButton on scroll up and show unread message count ([e554356](e554356)) * stop using FixedHeightMessage as default VirtualMessage component ([fc67915](fc67915)) * switch ladle to v2 ([ecd1cc6](ecd1cc6)) * **theming-v2:** add channel search for theme v2 ([#1685](#1685)) ([b735c30](b735c30)), closes [#1669](#1669) * **ThemingV2:** PopperTooltip component ([#1714](#1714)) ([9b6301e](9b6301e)) * **TypingIndicator:** add translations ([f079e26](f079e26)) * update @stream-io/stream-chat-css to v3.0.0 ([c5e392c](c5e392c)) * update message componets with theme v2 designs ([e5192d5](e5192d5)) * use FileIcon with version in UploadsPreview ([4d150b1](4d150b1)) * wrap ThreadHead content in a div to enable styling for class str-chat__parent-message-li ([9323edb](9323edb)) ### Reverts * fix: File attachment UI in theme-v1 ([789dd27](789dd27)) ### BREAKING CHANGES * ThemingV2 - user testing and adjustments * useMobilePress and useBreakpoint hooks are removed. useMobilePress: Historically, this hook programmatically handled the user interaction with Message components by toggling `mobile-press` class upon user interaction. The goal of this operation was to have the message actions displayed on the screen. Internally, we found a better solution by offloading this behavior to the browser and utilizing `:focus` and `:focus-within` CSS pseudo-selectors. useBreakpoint: This hook did hold the "programmatic" responsive UI breakpoints. We realized they aren't always in line with our stylesheet breakpoints and possibly with our customer's breakpoints. This misalignment was causing some inconsistencies and issues. We are removing this hook because we believe defining UI breakpoints should be responsibility of our customers. SearchResults: During the refactoring, we stumbled upon one side-effect where `popupResults` prop wasn't always respected. The fix of it could be a breaking change for a small percentage of our customers, but we believe this fix is the right thing to do.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🎯 Goal
Adjustments for the #1721 and #1714.