fix: replace FileReader with URL.createObjectURL #1701
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
arnautov-anton
force-pushed
the
fix/preview-uri
branch
from
33bd2b9 to
5330000
Compare
Codecov Report
@@ Coverage Diff @@
## develop #1701 +/- ##
===========================================
- Coverage 79.70% 79.68% -0.02%
===========================================
Files 229 229
Lines 5803 5795 -8
Branches 1713 1713
===========================================
- Hits 4625 4618 -7
+ Misses 1031 1029 -2
- Partials 147 148 +1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
oliverlaz
approved these changes
Aug 4, 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
Aug 29, 2022
# [9.5.0](v9.4.1...v9.5.0) (2022-08-29) ### Bug Fixes * include mdast-util-find-and-replace into our CJS bundle ([#1702](#1702)) ([61c4eec](61c4eec)), closes [#1698](#1698) * prevent double submissions in korean ([#1720](#1720)) ([5d781d8](5d781d8)) * replace FileReader with URL.createObjectURL ([#1701](#1701)) ([c8a490e](c8a490e)) * **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))
|
🎉 This PR is included in version 9.5.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
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
Fix image upload behaviour where preview URI does not get generated while the image is being uploaded.
🛠 Implementation details
Replace
FileReaderAPI withURL.createObjectURL.🎨 UI Changes