smig/excalidraw
1
0
Fork 0
mirror of https://github.com/excalidraw/excalidraw.git synced 2025-05-03 10:00:07 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

Use enum for color pickers instead of strings

Browse source
This commit is contained in:
Jared Palmer 2020-01-05 12:23:43 -08:00
parent 8d9ffb2f8a
commit 88e6948db0
1 changed files with 16 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

26
src/index.tsx
View file

@ -814,10 +814,16 @@ function restore(
}
}
enum ColorPicker {
CANVAS_BACKGROUND,
SHAPE_STROKE,
SHAPE_BACKGROUND
}
type AppState = {
draggingElement: ExcalidrawElement | null;
resizingElement: ExcalidrawElement | null;
currentColorPicker: "Background" | "ShapeStroke" | "ShapeBackground" | null;
currentColorPicker: ColorPicker | null;
elementType: string;
exportBackground: boolean;
currentItemStrokeColor: string;
@ -1165,13 +1171,13 @@ class App extends React.Component<{}, AppState> {
onClick={() =>
this.setState(s => ({
currentColorPicker:
s.currentColorPicker === "Background"
s.currentColorPicker === ColorPicker.CANVAS_BACKGROUND
? null
: "Background"
: ColorPicker.CANVAS_BACKGROUND
}))
}
></button>
{this.state.currentColorPicker === "Background" ? (
{this.state.currentColorPicker === ColorPicker.CANVAS_BACKGROUND ? (
<div className="popover">
<div
className="cover"
@ -1204,13 +1210,13 @@ class App extends React.Component<{}, AppState> {
onClick={() =>
this.setState(s => ({
currentColorPicker:
s.currentColorPicker === "ShapeStroke"
s.currentColorPicker === ColorPicker.SHAPE_STROKE
? null
: "ShapeStroke"
: ColorPicker.SHAPE_STROKE
}))
}
></button>
{this.state.currentColorPicker === "ShapeStroke" ? (
{this.state.currentColorPicker === ColorPicker.SHAPE_STROKE ? (
<div className="popover">
<div
className="cover"
@ -1243,13 +1249,13 @@ class App extends React.Component<{}, AppState> {
onClick={() =>
this.setState(s => ({
currentColorPicker:
s.currentColorPicker === "ShapeBackground"
s.currentColorPicker === ColorPicker.SHAPE_BACKGROUND
? null
: "ShapeBackground"
: ColorPicker.SHAPE_BACKGROUND
}))
}
></button>
{this.state.currentColorPicker === "ShapeBackground" ? (
{this.state.currentColorPicker === ColorPicker.SHAPE_BACKGROUND ? (
<div className="popover">
<div
className="cover"