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

fix: icon size being too small #309

Merged
merged 3 commits into from
Sep 5, 2022
Merged

fix: icon size being too small #309

merged 3 commits into from
Sep 5, 2022

Conversation

KaWaite
Copy link
Member

@KaWaite KaWaite commented Sep 5, 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 5, 2022

Deploy Preview for reearth-web ready!

Name Link
🔨 Latest commit 0b8fc98
🔍 Latest deploy log https://app.netlify.com/sites/reearth-web/deploys/63158c9266dd3100089ef929
😎 Deploy Preview https://deploy-preview-309--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 5, 2022

Codecov Report

Merging #309 (bfcddfc) into main (6a30f57) will not change coverage.
The diff coverage is 77.77%.

❗ Current head bfcddfc differs from pull request most recent head 0b8fc98. Consider uploading reports for the commit 0b8fc98 to get more accurate results

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #309   +/-   ##
=======================================
  Coverage   16.70%   16.70%           
=======================================
  Files         502      502           
  Lines       48691    48691           
  Branches      520      520           
=======================================
  Hits         8135     8135           
  Misses      40539    40539           
  Partials       17       17           
Impacted Files Coverage Δ
src/components/atoms/Button/index.tsx 0.00% <0.00%> (ø)
src/components/atoms/Icon/index.tsx 80.00% <87.50%> (ø)

@KaWaite KaWaite merged commit 3574f06 into main Sep 5, 2022
@KaWaite KaWaite deleted the fix/icons branch September 5, 2022 05:49
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