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

fix: icon button transition happening when undesired #313

Merged
merged 1 commit into from
Sep 8, 2022

Conversation

KaWaite
Copy link
Member

@KaWaite KaWaite commented Sep 8, 2022

Overview

What I've done

What I haven't done

How I tested

Screenshot

Which point I want you to review particularly

Memo

@netlify
Copy link

netlify bot commented Sep 8, 2022

Deploy Preview for reearth-web ready!

Name Link
🔨 Latest commit 4deecba
🔍 Latest deploy log https://app.netlify.com/sites/reearth-web/deploys/63197f942414fc00086489d1
😎 Deploy Preview https://deploy-preview-313--reearth-web.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@codecov
Copy link

codecov bot commented Sep 8, 2022

Codecov Report

Merging #313 (4deecba) into main (893d32f) will decrease coverage by 0.00%.
The diff coverage is 25.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #313      +/-   ##
==========================================
- Coverage   16.70%   16.70%   -0.01%     
==========================================
  Files         502      502              
  Lines       48704    48706       +2     
  Branches      520      520              
==========================================
  Hits         8135     8135              
- Misses      40552    40553       +1     
- Partials       17       18       +1     
Impacted Files Coverage Δ
src/components/atoms/Button/index.tsx 0.00% <0.00%> (ø)
src/components/atoms/Icon/index.tsx 79.12% <50.00%> (-0.88%) ⬇️

@KaWaite KaWaite merged commit 6a6d98e into main Sep 8, 2022
@KaWaite KaWaite deleted the fix/icon-transitions branch September 8, 2022 06:08
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.

1 participant