Skip to content
This repository has been archived by the owner on Apr 25, 2023. It is now read-only.

fix: widget bugs, language #89

Merged
merged 12 commits into from
Sep 30, 2021
Merged

fix: widget bugs, language #89

merged 12 commits into from
Sep 30, 2021

Conversation

KaWaite
Copy link
Member

@KaWaite KaWaite commented Sep 28, 2021

Overview

Couple bug fixes and a language update

What I've done

Fix uninstall plugin modal language.
Fix Menu(legacy) on mobile (button titles would get squished). Now buttons wrap.
Fix Storytelling many stories menu issue. (The more stories you added, the further the menu would move away from the widget's core.

Fix uninstall plugin modal language.
Fix Menu(legacy) on mobile.
Fix Storytelling many stories menu issue.
@KaWaite KaWaite requested review from HideBa and rot1024 September 28, 2021 02:25
@netlify
Copy link

netlify bot commented Sep 28, 2021

❌ Deploy Preview for reearth-web failed.

🔨 Explore the source changes: 1bdebbd

🔍 Inspect the deploy log: https://app.netlify.com/sites/reearth-web/deploys/61553b938f5f8200080283aa

@codecov
Copy link

codecov bot commented Sep 28, 2021

Codecov Report

Merging #89 (1bdebbd) into main (06cb14e) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #89   +/-   ##
=======================================
  Coverage   60.07%   60.07%           
=======================================
  Files          48       48           
  Lines         784      784           
  Branches      117      117           
=======================================
  Hits          471      471           
  Misses        263      263           
  Partials       50       50           

@rot1024 rot1024 changed the title Fix: widget bugs, language fix: widget bugs, language Sep 28, 2021
KaWaite and others added 2 commits September 28, 2021 11:41
HideBa
HideBa previously approved these changes Sep 28, 2021
Copy link
Member

@HideBa HideBa left a comment

Choose a reason for hiding this comment

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

LGTM

@rot1024 rot1024 merged commit 9de9dff into main Sep 30, 2021
@rot1024 rot1024 deleted the fix-09-27 branch September 30, 2021 04:28
keiya01 pushed a commit that referenced this pull request Apr 25, 2023
Co-authored-by: maherhamoui6 <[email protected]>
Co-authored-by: yk <[email protected]>
Co-authored-by: basel.issmail <[email protected]>
Co-authored-by: rot1024 <[email protected]>
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.

3 participants