Remove useMobilePress and useBreakpoint hooks #1648
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
oliverlaz
requested review from
MartinCupela,
arnautov-anton,
petyosi and
szuperaz
Codecov Report
@@ Coverage Diff @@
## theming-v2 #1648 +/- ##
==============================================
+ Coverage 79.51% 79.57% +0.06%
==============================================
Files 239 237 -2
Lines 5891 5851 -40
Branches 1651 1643 -8
==============================================
- Hits 4684 4656 -28
+ Misses 1060 1048 -12
Partials 147 147
Continue to review full report at Codecov.
|
|
I think if it's a breaking change we should have a note in the commit message body https://github.com/semantic-release/semantic-release#commit-message-format |
szuperaz
previously approved these changes
Jun 24, 2022
I agree, I can see that you have pretty sizable message in the first commit so you can extend it to contain |
Sure thing! I'll update the commit message accordingly 👍 |
oliverlaz
force-pushed
the
fix/remove-hooks
branch
from
1fdc97f to
91dc24c
Compare
oliverlaz
force-pushed
the
fix/remove-hooks
branch
2 times, most recently
from
e2b8695 to
4cb8c54
Compare
BREAKING CHANGE: 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 inline 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 small percentage of our customers, but we believe this fix is the right thing to do.
oliverlaz
force-pushed
the
fix/remove-hooks
branch
from
4cb8c54 to
12b4e91
Compare
oliverlaz
changed the title
MartinCupela
approved these changes
Jun 24, 2022
petyosi
approved these changes
Jun 27, 2022
petyosi
approved these changes
Jun 27, 2022
github-actions bot
pushed a commit
that referenced
this pull request
Aug 18, 2022
# [10.0.0-theming-v2.1](v9.4.0...v10.0.0-theming-v2.1) (2022-08-18) ### Bug Fixes * 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)) * 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)) * improve scrollToBottom with image attachments ([be8bb7a](be8bb7a)) * include mdast-util-find-and-replace into our CJS bundle ([#1702](#1702)) ([61c4eec](61c4eec)), closes [#1698](#1698) * **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)) * replace FileReader with URL.createObjectURL ([#1701](#1701)) ([c8a490e](c8a490e)) * Responsive layout ([7551650](7551650)) * **SuggestionList:** update trigger limits ([2e1f025](2e1f025)) * sync event listener keyDown type btw the image attachment and gallery modal ([51e7c14](51e7c14)) * **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 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)) * 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) * **TypingIndicator:** add translations ([f079e26](f079e26)) * 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.
|
🎉 This PR is included in version 10.0.0-theming-v2.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
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.
|
🎉 This PR is included in version 10.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
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
Remove useMobilePress and useBreakpoint hooks. Closes: #1641
🛠 Implementation details
useMobilePress:
Historically, this hook programmatically handled the user interaction with Message components by toggling
mobile-pressclass 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
:focusand:focus-withinCSS 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 the responsibility of our customers.
SearchResults:
During the refactoring, we stumbled upon one side-effect where
popupResultsprop 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.
Related PRs: