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

fix: unable to type RGBA values #180

Merged
merged 3 commits into from
Feb 28, 2022
Merged

fix: unable to type RGBA values #180

merged 3 commits into from
Feb 28, 2022

Conversation

KaWaite
Copy link
Member

@KaWaite KaWaite commented Feb 27, 2022

Overview

Fixes reearth/reearth-visualizer#233

What I've done

Fixed above bug where someone couldn't manually input RGBA values. Also refactored the code (created hooks file, reorganized function/constant declaration for better clarity).

@KaWaite KaWaite requested a review from rot1024 February 27, 2022 08:52
@netlify
Copy link

netlify bot commented Feb 27, 2022

✔️ Deploy Preview for reearth-web ready!

🔨 Explore the source changes: 4d1284b

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

😎 Browse the preview: https://deploy-preview-180--reearth-web.netlify.app

Copy link
Member

@rot1024 rot1024 left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov
Copy link

codecov bot commented Feb 27, 2022

Codecov Report

Merging #180 (4d1284b) into main (b0df23a) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #180   +/-   ##
=======================================
  Coverage   45.14%   45.14%           
=======================================
  Files          60       60           
  Lines        1309     1309           
  Branches      212      212           
=======================================
  Hits          591      591           
  Misses        659      659           
  Partials       59       59           

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.

What I reviewed might not be what you changed. Just in case they might cause bug, I left comments. Kindly check. 👍

@KaWaite KaWaite merged commit f7345c5 into main Feb 28, 2022
@KaWaite KaWaite deleted the fix-infobox-bg branch February 28, 2022 03:17
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.

Entering a value for the background color of the infobox will not be applied.
3 participants