aws-samples / bedrock-claude-chat

AWS-native chatbot using Bedrock + Claude (+Mistral)
MIT No Attribution
784 stars 284 forks source link

[Feature Request] Support Safari and Edge #170

Closed imyoungyang closed 4 months ago

imyoungyang commented 5 months ago

Describe the solution you'd like If run on Safari, it will get the error: Unexpected Application Error! D.checkVisibility is not a function. (In 'D.checkVisibility()', 'D.checkVisibility' is undefined)

Why the solution needed A clear and concise description of the reason why you want the solution.

Additional context Add any other context or screenshots about the feature request here.

kyosuke-kobayashi-lvgs commented 5 months ago

frontend/src/components/ChatListDrawer.tsx

useLayoutEffect内で計算されている箇所が影響していそうです It seems to be affected by the part of the calculation in useLayoutEffect.

kyosuke-kobayashi-lvgs commented 5 months ago

FYI : https://caniuse.com/mdn-api_element_checkvisibility

kyosuke-kobayashi-lvgs commented 5 months ago

It seems to be supported in safari 17.4

safari release notes https://developer.apple.com/documentation/safari-release-notes/safari-17_4-release-notes