caseycesari Github contribution chart
caseycesari Github Stats
caseycesari Most Used Languages

Activity

14 Sep 2022

Issue Comment

Caseycesari

Add labels to municipal boundaries layer

Overview

This PR adds labels to the municipal boundaries layer.

If the zoom level is less than 9, the labels are not shown as this gets too busy. On the satellite basemap, the labels are white text instead of the default black.

Closes #49

Demo

Testing Instructions

  • Go to draw map page
  • Enable municipal boundaries layer
  • Zoom in past zoom level 9
  • Labels should be present
  • Switch to satellite basemap
  • The labels should now be white text

Checklist

  • [ ] fixup! commits have been squashed
  • [ ] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines
  • [ ] README.md updated if necessary to reflect the changes
  • [ ] CI passes after rebase

Forked On 14 Sep 2022 at 04:45:31

Caseycesari

That looks good to me too!

Commented On 14 Sep 2022 at 04:45:31

Caseycesari

Drop Vagrant, just use Docker directly

Previously Vagrant was usable in more places - nowadays Docker should be usable anywhere.

Testing Instructions

  • Confirm it works for a Mac user.

Merged On 08 Sep 2022 at 04:14:56

Caseycesari

Ran ./scripts/update followed by ./scripts/server. The build succeeded and I got a functioning development site:

image

Commented On 08 Sep 2022 at 04:14:56
Issue Comment

Caseycesari

Add Parcel Layer

Overview

Adds Parcel Layer via the NC One Map ArcGIS API.

Closes #33

Demo

~~https://user-images.githubusercontent.com/1430060/187294794-54343556-8989-4515-a6cd-8ca3f50b9638.mp4~~ Old demo. Replaced with:

https://user-images.githubusercontent.com/1430060/187306338-8a1ddc78-346d-4c60-aaea-c27fba97947b.mp4

Notes

~~The performance, despite all the tweaks, is quite dismal.~~ Improved in 952496a and 36d5585

~~One avenue to investigate may be to fetch images from the MapServer rather than features from the FeatureServer.~~ Implemented in 952496a.

~~Another improvement we can make is to show the Parcel Centroids, which are also available as a layer, at lower zoom levels. This will give users something rather than nothing.~~ Implemented in 1c7c41c.

Testing Instructions

  • Check out this branch and server
  • Go to http://localhost:4545/draw
    • [ ] Ensure you see a disabled Parcel data layer button in the sidebar
  • Zoom in to the map
    • [ ] Ensure the button turns on once you are zoomed in enough
    • [ ] Ensure the layer appears, as styled in Figma, when you are zoomed in enough
    • [ ] Ensure you can toggle the layer on and off while zoomed in enough

Checklist

  • [ ] fixup! commits have been squashed
  • [ ] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines
  • [ ] README.md updated if necessary to reflect the changes
  • [ ] CI passes after rebase

Forked On 30 Aug 2022 at 01:00:51

Caseycesari

@rajadain Nice job figuring out a way to improve the performance! I'm glad this worked out because I wasn't looking forward to maintaining our own parcel layer.

Commented On 30 Aug 2022 at 01:00:51

Caseycesari

Add Parcel Layer

Overview

Adds Parcel Layer via the NC One Map ArcGIS API.

Closes #33

Demo

~~https://user-images.githubusercontent.com/1430060/187294794-54343556-8989-4515-a6cd-8ca3f50b9638.mp4~~ Old demo. Replaced with:

https://user-images.githubusercontent.com/1430060/187306338-8a1ddc78-346d-4c60-aaea-c27fba97947b.mp4

Notes

~~The performance, despite all the tweaks, is quite dismal.~~ Improved in 952496a and 36d5585

~~One avenue to investigate may be to fetch images from the MapServer rather than features from the FeatureServer.~~ Implemented in 952496a.

~~Another improvement we can make is to show the Parcel Centroids, which are also available as a layer, at lower zoom levels. This will give users something rather than nothing.~~ Implemented in 1c7c41c.

Testing Instructions

  • Check out this branch and server
  • Go to http://localhost:4545/draw
    • [ ] Ensure you see a disabled Parcel data layer button in the sidebar
  • Zoom in to the map
    • [ ] Ensure the button turns on once you are zoomed in enough
    • [ ] Ensure the layer appears, as styled in Figma, when you are zoomed in enough
    • [ ] Ensure you can toggle the layer on and off while zoomed in enough

Checklist

  • [ ] fixup! commits have been squashed
  • [ ] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines
  • [ ] README.md updated if necessary to reflect the changes
  • [ ] CI passes after rebase

Forked On 30 Aug 2022 at 12:59:41

Caseycesari

Curios for @gatesgodin input, but I feel like the parcel centroids layer won't be useful for this application.
On 30 Aug 2022 at 12:59:41

Caseycesari

Add Parcel Layer

Overview

Adds Parcel Layer via the NC One Map ArcGIS API.

Closes #33

Demo

~~https://user-images.githubusercontent.com/1430060/187294794-54343556-8989-4515-a6cd-8ca3f50b9638.mp4~~ Old demo. Replaced with:

https://user-images.githubusercontent.com/1430060/187306338-8a1ddc78-346d-4c60-aaea-c27fba97947b.mp4

Notes

~~The performance, despite all the tweaks, is quite dismal.~~ Improved in 952496a and 36d5585

~~One avenue to investigate may be to fetch images from the MapServer rather than features from the FeatureServer.~~ Implemented in 952496a.

~~Another improvement we can make is to show the Parcel Centroids, which are also available as a layer, at lower zoom levels. This will give users something rather than nothing.~~ Implemented in 1c7c41c.

Testing Instructions

  • Check out this branch and server
  • Go to http://localhost:4545/draw
    • [ ] Ensure you see a disabled Parcel data layer button in the sidebar
  • Zoom in to the map
    • [ ] Ensure the button turns on once you are zoomed in enough
    • [ ] Ensure the layer appears, as styled in Figma, when you are zoomed in enough
    • [ ] Ensure you can toggle the layer on and off while zoomed in enough

Checklist

  • [ ] fixup! commits have been squashed
  • [ ] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines
  • [ ] README.md updated if necessary to reflect the changes
  • [ ] CI passes after rebase

Merged On 30 Aug 2022 at 12:59:41

Caseycesari

Commented On 30 Aug 2022 at 12:59:41
Issue Comment

Caseycesari

Add Parcel Data Layer

Overview

  • [ ] Find out which Parcel Data layer we're using
  • [ ] Decide if it can be queried from a remote source or if it needs to be ingested and tiled
    • [ ] If needed, ingest it
    • [ ] Tile it
  • [ ] Add to map, wire up to Parcel data button under Basemap Layers

Forked On 29 Aug 2022 at 04:31:42

Caseycesari

Parcel data can be found here in various formats https://www.nconemap.gov/pages/parcels

Commented On 29 Aug 2022 at 04:31:42
Issue Comment

Caseycesari

Add Topographic basemap for Land & water layer

Overview

Since we can't add individual vector tiles, we switch the basemap wholesale. When the Land & water layer is enabled, and the default basemap is set to active, we show the Topographic layer. Otherwise we show the Light Grey Canvas layer. The Satellite tile shows that layer regardless of Land & water state.

Closes #47

Demo

https://user-images.githubusercontent.com/1430060/187044736-63a2ea79-6927-428d-8d48-d1ee0e6d9c25.mp4

Testing Instructions

  • Checkout this branch and server
  • Go to http://localhost:4545/draw
  • Toggle the Land & water layer
    • [ ] Ensure you see the Topographic basemap when the layer is on and the basemap tile is Default
    • [ ] Ensure you see the Light Grey Canvas basemap when the layer is off
    • [ ] Ensure you see the Satellite basemap when the tile is Satellite, regardless of state of Land & water layer

Checklist

  • [ ] fixup! commits have been squashed
  • [ ] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines
  • [ ] README.md updated if necessary to reflect the changes
  • [ ] CI passes after rebase

Forked On 29 Aug 2022 at 03:42:22

Caseycesari

One quick things that are not directly related to this PR, but that could be addressed here just for efficiency's sake: can the HTML app name be switched from React App to Boundary Sync? image

Commented On 29 Aug 2022 at 03:42:22
Issue Comment

Caseycesari

Add municipal boundaries layer

Overview

This PR adds the municipal boundaries layer. The data is sourced from https://linc.osbm.nc.gov/explore/dataset/municipalities-2020/.

The GeoJSON file is compressed with weback and then dynamically imported using the technique described here: https://www.azavea.com/blog/2022/04/28/how-to-code-split-geojson-layers-for-your-mapboxgl-js-react-apps/.

Closes #34

Notes

The GeoJSON file compresses to about 4mb. If this is too much or the rendering of the entire file causes choppiness, then an alternative approach might be needed. A possibility is to split the layer into tiles and serve it from AWS.

Testing Instructions

  • Go to the draw page
  • The municipal boundaries shoud show up.
  • Clicking the sidebar visibility button should toggle the visibility of the layer

Checklist

  • [ ] fixup! commits have been squashed
  • [x] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines
  • [x] README.md updated if necessary to reflect the changes
  • [ ] CI passes after rebase

Forked On 26 Aug 2022 at 11:13:16

Caseycesari

@rajadain You may need to run ./scripts/fetch-data again to rename the file and put it in the new spot.

Commented On 26 Aug 2022 at 11:13:16
Issue Comment

Caseycesari

Add Parcel Data Layer

Overview

  • [ ] Find out which Parcel Data layer we're using
  • [ ] Decide if it can be queried from a remote source or if it needs to be ingested and tiled
    • [ ] If needed, ingest it
    • [ ] Tile it
  • [ ] Add to map, wire up to Parcel data button under Basemap Layers

Forked On 19 Aug 2022 at 09:53:39

Caseycesari

Blocked until we receive the parcel data.

Commented On 19 Aug 2022 at 09:53:39

Caseycesari

Draw Map User Interface

Overview

This PR adds the UI elements to the Draw page map. It also adds some basic polygon functionality (without using a global store/state).

Closes #24

Notes

As of now, the polygon edits don't persist when toggling visibility/edit mode. This is because there is no global store for this data.

To Do

As part of this PR:

  • [ ] Center modal/dialogs over map, rather than whole screen
  • [x] Prevent double click from recentering map when clicking on other elements

Possibly a future PR:

  • [ ] Search features
  • [ ] Basemap and basemap layers

Definitely a future PR:

  • [ ] Add state/store so polygon data will persist. This will require a bit of refactoring.

Testing Instructions

  • Go to http://localhost:4545/draw
  • Click "Draw Polygon": a square should appear in the center of the map
  • Click "New Polygon": a modal should pop up that allows you to change the name of the polygon
  • Click "Show/Hide": this should toggle the visibility of the polygon
  • Click "Edit Points": this should toggle whether the point drag handles are visible and usable
  • Click "Delete Polygon": after confirmation, this should delete the polygon
  • The polygon should be editable by dragging midpoints to create new verticies, or dragging corners to move them.
  • The "Zoom In" and "Zoom Out" buttons should zoom in and out respectively

Checklist

  • [ ] fixup! commits have been squashed
  • [ ] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines
  • [ ] README.md updated if necessary to reflect the changes
  • [ ] CI passes after rebase

Forked On 19 Aug 2022 at 09:36:56

Caseycesari

Nice use of hooks.
On 19 Aug 2022 at 09:36:56

Caseycesari

Draw Map User Interface

Overview

This PR adds the UI elements to the Draw page map. It also adds some basic polygon functionality (without using a global store/state).

Closes #24

Notes

As of now, the polygon edits don't persist when toggling visibility/edit mode. This is because there is no global store for this data.

To Do

As part of this PR:

  • [ ] Center modal/dialogs over map, rather than whole screen
  • [x] Prevent double click from recentering map when clicking on other elements

Possibly a future PR:

  • [ ] Search features
  • [ ] Basemap and basemap layers

Definitely a future PR:

  • [ ] Add state/store so polygon data will persist. This will require a bit of refactoring.

Testing Instructions

  • Go to http://localhost:4545/draw
  • Click "Draw Polygon": a square should appear in the center of the map
  • Click "New Polygon": a modal should pop up that allows you to change the name of the polygon
  • Click "Show/Hide": this should toggle the visibility of the polygon
  • Click "Edit Points": this should toggle whether the point drag handles are visible and usable
  • Click "Delete Polygon": after confirmation, this should delete the polygon
  • The polygon should be editable by dragging midpoints to create new verticies, or dragging corners to move them.
  • The "Zoom In" and "Zoom Out" buttons should zoom in and out respectively

Checklist

  • [ ] fixup! commits have been squashed
  • [ ] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines
  • [ ] README.md updated if necessary to reflect the changes
  • [ ] CI passes after rebase

Merged On 19 Aug 2022 at 09:48:39

Caseycesari

This is really well done. Nice work!

Commented On 19 Aug 2022 at 09:48:39

Caseycesari

Draw Map User Interface

Overview

This PR adds the UI elements to the Draw page map. It also adds some basic polygon functionality (without using a global store/state).

Closes #24

Notes

As of now, the polygon edits don't persist when toggling visibility/edit mode. This is because there is no global store for this data.

To Do

As part of this PR:

  • [ ] Center modal/dialogs over map, rather than whole screen
  • [x] Prevent double click from recentering map when clicking on other elements

Possibly a future PR:

  • [ ] Search features
  • [ ] Basemap and basemap layers

Definitely a future PR:

  • [ ] Add state/store so polygon data will persist. This will require a bit of refactoring.

Testing Instructions

  • Go to http://localhost:4545/draw
  • Click "Draw Polygon": a square should appear in the center of the map
  • Click "New Polygon": a modal should pop up that allows you to change the name of the polygon
  • Click "Show/Hide": this should toggle the visibility of the polygon
  • Click "Edit Points": this should toggle whether the point drag handles are visible and usable
  • Click "Delete Polygon": after confirmation, this should delete the polygon
  • The polygon should be editable by dragging midpoints to create new verticies, or dragging corners to move them.
  • The "Zoom In" and "Zoom Out" buttons should zoom in and out respectively

Checklist

  • [ ] fixup! commits have been squashed
  • [ ] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines
  • [ ] README.md updated if necessary to reflect the changes
  • [ ] CI passes after rebase

Forked On 19 Aug 2022 at 09:47:40

Caseycesari

Nice job figuring out a way to do override the marker styles.
On 19 Aug 2022 at 09:47:40

Caseycesari

Draw Map User Interface

Overview

This PR adds the UI elements to the Draw page map. It also adds some basic polygon functionality (without using a global store/state).

Closes #24

Notes

As of now, the polygon edits don't persist when toggling visibility/edit mode. This is because there is no global store for this data.

To Do

As part of this PR:

  • [ ] Center modal/dialogs over map, rather than whole screen
  • [x] Prevent double click from recentering map when clicking on other elements

Possibly a future PR:

  • [ ] Search features
  • [ ] Basemap and basemap layers

Definitely a future PR:

  • [ ] Add state/store so polygon data will persist. This will require a bit of refactoring.

Testing Instructions

  • Go to http://localhost:4545/draw
  • Click "Draw Polygon": a square should appear in the center of the map
  • Click "New Polygon": a modal should pop up that allows you to change the name of the polygon
  • Click "Show/Hide": this should toggle the visibility of the polygon
  • Click "Edit Points": this should toggle whether the point drag handles are visible and usable
  • Click "Delete Polygon": after confirmation, this should delete the polygon
  • The polygon should be editable by dragging midpoints to create new verticies, or dragging corners to move them.
  • The "Zoom In" and "Zoom Out" buttons should zoom in and out respectively

Checklist

  • [ ] fixup! commits have been squashed
  • [ ] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines
  • [ ] README.md updated if necessary to reflect the changes
  • [ ] CI passes after rebase

Merged On 19 Aug 2022 at 09:48:39

Caseycesari

This is really well done. Nice work!

Commented On 19 Aug 2022 at 09:48:39
Issue Comment

Caseycesari

Add Municipal Boundaries Layer

Overview

  • [ ] Find out which Municipal Boundaries layer we're using
  • [ ] Decide if it can be queried from a remote source or if it needs to be ingested and tiled
    • [ ] If needed, ingest it
    • [ ] Tile it
  • [ ] Add to map, wire up to Municipal Boundaries button under Basemap Layers

Forked On 18 Aug 2022 at 07:34:46

Caseycesari

Possible data source https://linc.osbm.nc.gov/explore/dataset/municipalities-2020/information/?disjunctive.name20&location=6,36.96224,-74.88288&basemap=jawg.streets&dataChart=eyJxdWVyaWVzIjpbeyJjb25maWciOnsiZGF0YXNldCI6Im11bmljaXBhbGl0aWVzLTIwMjAiLCJvcHRpb25zIjp7ImRpc2p1bmN0aXZlLm5hbWUyMCI6dHJ1ZSwiYmFzZW1hcCI6Imphd2cuc3RyZWV0cyIsImxvY2F0aW9uIjoiNywzNS4yMDYzLC03OS44MDM3NCJ9fSwiY2hhcnRzIjpbeyJhbGlnbk1vbnRoIjp0cnVlLCJ0eXBlIjoibGluZSIsImZ1bmMiOiJBVkciLCJ5QXhpcyI6ImxhbmRhcmVhIiwic2NpZW50aWZpY0Rpc3BsYXkiOnRydWUsImNvbG9yIjoiIzQ4N0FBOCJ9XSwieEF4aXMiOiJ5ZWFyX2luY29ycG9yYXRlZCIsIm1heHBvaW50cyI6IiIsInRpbWVzY2FsZSI6InllYXIiLCJzb3J0IjoiIn1dLCJkaXNwbGF5TGVnZW5kIjp0cnVlLCJhbGlnbk1vbnRoIjp0cnVlfQ%3D%3D

Commented On 18 Aug 2022 at 07:34:46
Issue Comment

Caseycesari

error TS2456: Type alias 'Geometry' circularly references itself.

Suddenly getting an error while trying to build an angular component with this library: ERROR in node_modules/@types/geojson/index.d.ts(79,13): error TS2456: Type alias 'Geometry' circularly references itself.

using https://registry.npmjs.org/@types/geojson/-/geojson-7946.0.10.tgz

Forked On 18 Aug 2022 at 09:52:43

Caseycesari

Thanks @holdenparker for point @aninaA in the right direction. Going to close this out.

Commented On 18 Aug 2022 at 09:52:43

Caseycesari

App Template Updates

Overview

This PR incorporates changes made to the app template into this project.

Closes #28

Testing Instructions

  • ./scripts/setup
  • ./scripts/update
  • ./scripts/server
  • Browse around and check things are working as expected

Checklist

  • [x] fixup! commits have been squashed
  • [x] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines
  • [ ] README.md updated if necessary to reflect the changes
  • [ ] CI passes after rebase

Forked On 17 Aug 2022 at 07:23:33

Caseycesari

Sorry, should have specified this in the issue, but this the default app template Django port. It's been changed to `8181` in this project, and that should be used here.
On 17 Aug 2022 at 07:23:33

Caseycesari

App Template Updates

Overview

This PR incorporates changes made to the app template into this project.

Closes #28

Testing Instructions

  • ./scripts/setup
  • ./scripts/update
  • ./scripts/server
  • Browse around and check things are working as expected

Checklist

  • [x] fixup! commits have been squashed
  • [x] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines
  • [ ] README.md updated if necessary to reflect the changes
  • [ ] CI passes after rebase

Merged On 17 Aug 2022 at 08:51:02

Caseycesari

One small change otherwise this looks good!

Commented On 17 Aug 2022 at 08:51:02

Caseycesari

App Template Updates

Overview

This PR incorporates changes made to the app template into this project.

Closes #28

Testing Instructions

  • ./scripts/setup
  • ./scripts/update
  • ./scripts/server
  • Browse around and check things are working as expected

Checklist

  • [x] fixup! commits have been squashed
  • [x] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines
  • [ ] README.md updated if necessary to reflect the changes
  • [ ] CI passes after rebase

Merged On 17 Aug 2022 at 08:51:02

Caseycesari

One small change otherwise this looks good!

Commented On 17 Aug 2022 at 08:51:02

Caseycesari

Add Prototype Skeleton

Overview

This adds 2 base pages: http://localhost:4545/welcome and http://localhost:4545/main which correspond to the Welcome Modal and Map and Sidebar pages respectively, allowing them to be developed independently.

The Welcome Model has a Leaflet Map using ESRI's Raster Light Gray Canvas basemap centered to North Carolina in the background, with a non-dismissable Chakra UI Modal in the foreground. This corresponds to the Welcome Flow wireframes on Figma.

The Map and Sidebar page uses the same map, with a sidebar. This layout uses Chakra's Flex component.

Once this is in, we should make cards for the individual sub-pages. A reasonable attempt should be made to match the wireframes, in terms of colors / spacing / fonts.

Thanks to @mstone121, @jacobtylerwalls, and @rachelekm for their input in this work.

Closes #15

Demo

image

image

Notes

Seeing this warning in the Webpack output, will address this in a future PR:

WARNING in ./node_modules/leaflet/dist/leaflet.css (./node_modules/css-loader/dist/cjs.js??ruleSet[1].rules[2].oneOf[5].use[1]!./node_modules/postcss-loader/dist/cjs.js??ruleSet[1].rules[2].oneOf[5].use[2]!./node_modules/source-map-loader/dist/cjs.js!./node_modules/leaflet/dist/leaflet.css)
Module Warning (from ./node_modules/postcss-loader/dist/cjs.js):
Warning

(655:3) autoprefixer: Replace color-adjust to print-color-adjust. The color-adjust shorthand is currently deprecated.

webpack compiled with 1 warning
Compiling...
Compiled with warnings.

Warning
(655:3) autoprefixer: Replace color-adjust to print-color-adjust. The color-adjust shorthand is currently deprecated.

Search for the keywords to learn more about each warning.
To ignore, add // eslint-disable-next-line to the line before 

Testing Instructions

  • Check out this branch and update and server
  • Go to http://localhost:4545/
    • [ ] Ensure you are redirected to http://localhost:4545/welcome
    • [ ] Ensure you see the grayscale map in the background, with a modal on top
  • Go to http://localhost:4545/main
    • [ ] Ensure you see a blue-gray sidebar next to the map
  • Inspect the JavaScript console
    • [ ] Ensure there are no errors

Forked On 11 Aug 2022 at 05:32:12

Caseycesari

That makes sense. I think in the past I have just updated the `package.json` file directly and then ran `./scripts/update`, but I think editing the package.json directly is kind of frowned upon.
On 11 Aug 2022 at 05:32:12

Caseycesari

Add Prototype Skeleton

Overview

This adds 2 base pages: http://localhost:4545/welcome and http://localhost:4545/main which correspond to the Welcome Modal and Map and Sidebar pages respectively, allowing them to be developed independently.

The Welcome Model has a Leaflet Map using ESRI's Raster Light Gray Canvas basemap centered to North Carolina in the background, with a non-dismissable Chakra UI Modal in the foreground. This corresponds to the Welcome Flow wireframes on Figma.

The Map and Sidebar page uses the same map, with a sidebar. This layout uses Chakra's Flex component.

Once this is in, we should make cards for the individual sub-pages. A reasonable attempt should be made to match the wireframes, in terms of colors / spacing / fonts.

Thanks to @mstone121, @jacobtylerwalls, and @rachelekm for their input in this work.

Closes #15

Demo

image

image

Notes

Seeing this warning in the Webpack output, will address this in a future PR:

WARNING in ./node_modules/leaflet/dist/leaflet.css (./node_modules/css-loader/dist/cjs.js??ruleSet[1].rules[2].oneOf[5].use[1]!./node_modules/postcss-loader/dist/cjs.js??ruleSet[1].rules[2].oneOf[5].use[2]!./node_modules/source-map-loader/dist/cjs.js!./node_modules/leaflet/dist/leaflet.css)
Module Warning (from ./node_modules/postcss-loader/dist/cjs.js):
Warning

(655:3) autoprefixer: Replace color-adjust to print-color-adjust. The color-adjust shorthand is currently deprecated.

webpack compiled with 1 warning
Compiling...
Compiled with warnings.

Warning
(655:3) autoprefixer: Replace color-adjust to print-color-adjust. The color-adjust shorthand is currently deprecated.

Search for the keywords to learn more about each warning.
To ignore, add // eslint-disable-next-line to the line before 

Testing Instructions

  • Check out this branch and update and server
  • Go to http://localhost:4545/
    • [ ] Ensure you are redirected to http://localhost:4545/welcome
    • [ ] Ensure you see the grayscale map in the background, with a modal on top
  • Go to http://localhost:4545/main
    • [ ] Ensure you see a blue-gray sidebar next to the map
  • Inspect the JavaScript console
    • [ ] Ensure there are no errors

Merged On 11 Aug 2022 at 05:32:13

Caseycesari

Commented On 11 Aug 2022 at 05:32:13

Caseycesari

Add Prototype Skeleton

Overview

This adds 2 base pages: http://localhost:4545/welcome and http://localhost:4545/main which correspond to the Welcome Modal and Map and Sidebar pages respectively, allowing them to be developed independently.

The Welcome Model has a Leaflet Map using ESRI's Raster Light Gray Canvas basemap centered to North Carolina in the background, with a non-dismissable Chakra UI Modal in the foreground. This corresponds to the Welcome Flow wireframes on Figma.

The Map and Sidebar page uses the same map, with a sidebar. This layout uses Chakra's Flex component.

Once this is in, we should make cards for the individual sub-pages. A reasonable attempt should be made to match the wireframes, in terms of colors / spacing / fonts.

Thanks to @mstone121, @jacobtylerwalls, and @rachelekm for their input in this work.

Closes #15

Demo

image

image

Notes

Seeing this warning in the Webpack output, will address this in a future PR:

WARNING in ./node_modules/leaflet/dist/leaflet.css (./node_modules/css-loader/dist/cjs.js??ruleSet[1].rules[2].oneOf[5].use[1]!./node_modules/postcss-loader/dist/cjs.js??ruleSet[1].rules[2].oneOf[5].use[2]!./node_modules/source-map-loader/dist/cjs.js!./node_modules/leaflet/dist/leaflet.css)
Module Warning (from ./node_modules/postcss-loader/dist/cjs.js):
Warning

(655:3) autoprefixer: Replace color-adjust to print-color-adjust. The color-adjust shorthand is currently deprecated.

webpack compiled with 1 warning
Compiling...
Compiled with warnings.

Warning
(655:3) autoprefixer: Replace color-adjust to print-color-adjust. The color-adjust shorthand is currently deprecated.

Search for the keywords to learn more about each warning.
To ignore, add // eslint-disable-next-line to the line before 

Testing Instructions

  • Check out this branch and update and server
  • Go to http://localhost:4545/
    • [ ] Ensure you are redirected to http://localhost:4545/welcome
    • [ ] Ensure you see the grayscale map in the background, with a modal on top
  • Go to http://localhost:4545/main
    • [ ] Ensure you see a blue-gray sidebar next to the map
  • Inspect the JavaScript console
    • [ ] Ensure there are no errors

Forked On 11 Aug 2022 at 04:53:47

Caseycesari

Minor thing, but consider renaming this to `draw` or something like to indicate the primary purpose of the page.
On 11 Aug 2022 at 04:53:47

Caseycesari

Add Prototype Skeleton

Overview

This adds 2 base pages: http://localhost:4545/welcome and http://localhost:4545/main which correspond to the Welcome Modal and Map and Sidebar pages respectively, allowing them to be developed independently.

The Welcome Model has a Leaflet Map using ESRI's Raster Light Gray Canvas basemap centered to North Carolina in the background, with a non-dismissable Chakra UI Modal in the foreground. This corresponds to the Welcome Flow wireframes on Figma.

The Map and Sidebar page uses the same map, with a sidebar. This layout uses Chakra's Flex component.

Once this is in, we should make cards for the individual sub-pages. A reasonable attempt should be made to match the wireframes, in terms of colors / spacing / fonts.

Thanks to @mstone121, @jacobtylerwalls, and @rachelekm for their input in this work.

Closes #15

Demo

image

image

Notes

Seeing this warning in the Webpack output, will address this in a future PR:

WARNING in ./node_modules/leaflet/dist/leaflet.css (./node_modules/css-loader/dist/cjs.js??ruleSet[1].rules[2].oneOf[5].use[1]!./node_modules/postcss-loader/dist/cjs.js??ruleSet[1].rules[2].oneOf[5].use[2]!./node_modules/source-map-loader/dist/cjs.js!./node_modules/leaflet/dist/leaflet.css)
Module Warning (from ./node_modules/postcss-loader/dist/cjs.js):
Warning

(655:3) autoprefixer: Replace color-adjust to print-color-adjust. The color-adjust shorthand is currently deprecated.

webpack compiled with 1 warning
Compiling...
Compiled with warnings.

Warning
(655:3) autoprefixer: Replace color-adjust to print-color-adjust. The color-adjust shorthand is currently deprecated.

Search for the keywords to learn more about each warning.
To ignore, add // eslint-disable-next-line to the line before 

Testing Instructions

  • Check out this branch and update and server
  • Go to http://localhost:4545/
    • [ ] Ensure you are redirected to http://localhost:4545/welcome
    • [ ] Ensure you see the grayscale map in the background, with a modal on top
  • Go to http://localhost:4545/main
    • [ ] Ensure you see a blue-gray sidebar next to the map
  • Inspect the JavaScript console
    • [ ] Ensure there are no errors

Forked On 11 Aug 2022 at 04:55:28

Caseycesari

No preference on yarn vs npm from me, but just curious of an example command we would be using this script for.
On 11 Aug 2022 at 04:55:28

Caseycesari

Add Prototype Skeleton

Overview

This adds 2 base pages: http://localhost:4545/welcome and http://localhost:4545/main which correspond to the Welcome Modal and Map and Sidebar pages respectively, allowing them to be developed independently.

The Welcome Model has a Leaflet Map using ESRI's Raster Light Gray Canvas basemap centered to North Carolina in the background, with a non-dismissable Chakra UI Modal in the foreground. This corresponds to the Welcome Flow wireframes on Figma.

The Map and Sidebar page uses the same map, with a sidebar. This layout uses Chakra's Flex component.

Once this is in, we should make cards for the individual sub-pages. A reasonable attempt should be made to match the wireframes, in terms of colors / spacing / fonts.

Thanks to @mstone121, @jacobtylerwalls, and @rachelekm for their input in this work.

Closes #15

Demo

image

image

Notes

Seeing this warning in the Webpack output, will address this in a future PR:

WARNING in ./node_modules/leaflet/dist/leaflet.css (./node_modules/css-loader/dist/cjs.js??ruleSet[1].rules[2].oneOf[5].use[1]!./node_modules/postcss-loader/dist/cjs.js??ruleSet[1].rules[2].oneOf[5].use[2]!./node_modules/source-map-loader/dist/cjs.js!./node_modules/leaflet/dist/leaflet.css)
Module Warning (from ./node_modules/postcss-loader/dist/cjs.js):
Warning

(655:3) autoprefixer: Replace color-adjust to print-color-adjust. The color-adjust shorthand is currently deprecated.

webpack compiled with 1 warning
Compiling...
Compiled with warnings.

Warning
(655:3) autoprefixer: Replace color-adjust to print-color-adjust. The color-adjust shorthand is currently deprecated.

Search for the keywords to learn more about each warning.
To ignore, add // eslint-disable-next-line to the line before 

Testing Instructions

  • Check out this branch and update and server
  • Go to http://localhost:4545/
    • [ ] Ensure you are redirected to http://localhost:4545/welcome
    • [ ] Ensure you see the grayscale map in the background, with a modal on top
  • Go to http://localhost:4545/main
    • [ ] Ensure you see a blue-gray sidebar next to the map
  • Inspect the JavaScript console
    • [ ] Ensure there are no errors

Merged On 11 Aug 2022 at 04:55:33

Caseycesari

Commented On 11 Aug 2022 at 04:55:33

Caseycesari

Add Prototype Skeleton

Overview

This adds 2 base pages: http://localhost:4545/welcome and http://localhost:4545/main which correspond to the Welcome Modal and Map and Sidebar pages respectively, allowing them to be developed independently.

The Welcome Model has a Leaflet Map using ESRI's Raster Light Gray Canvas basemap centered to North Carolina in the background, with a non-dismissable Chakra UI Modal in the foreground. This corresponds to the Welcome Flow wireframes on Figma.

The Map and Sidebar page uses the same map, with a sidebar. This layout uses Chakra's Flex component.

Once this is in, we should make cards for the individual sub-pages. A reasonable attempt should be made to match the wireframes, in terms of colors / spacing / fonts.

Thanks to @mstone121, @jacobtylerwalls, and @rachelekm for their input in this work.

Closes #15

Demo

image

image

Notes

Seeing this warning in the Webpack output, will address this in a future PR:

WARNING in ./node_modules/leaflet/dist/leaflet.css (./node_modules/css-loader/dist/cjs.js??ruleSet[1].rules[2].oneOf[5].use[1]!./node_modules/postcss-loader/dist/cjs.js??ruleSet[1].rules[2].oneOf[5].use[2]!./node_modules/source-map-loader/dist/cjs.js!./node_modules/leaflet/dist/leaflet.css)
Module Warning (from ./node_modules/postcss-loader/dist/cjs.js):
Warning

(655:3) autoprefixer: Replace color-adjust to print-color-adjust. The color-adjust shorthand is currently deprecated.

webpack compiled with 1 warning
Compiling...
Compiled with warnings.

Warning
(655:3) autoprefixer: Replace color-adjust to print-color-adjust. The color-adjust shorthand is currently deprecated.

Search for the keywords to learn more about each warning.
To ignore, add // eslint-disable-next-line to the line before 

Testing Instructions

  • Check out this branch and update and server
  • Go to http://localhost:4545/
    • [ ] Ensure you are redirected to http://localhost:4545/welcome
    • [ ] Ensure you see the grayscale map in the background, with a modal on top
  • Go to http://localhost:4545/main
    • [ ] Ensure you see a blue-gray sidebar next to the map
  • Inspect the JavaScript console
    • [ ] Ensure there are no errors

Merged On 11 Aug 2022 at 04:55:32

Caseycesari

Commented On 11 Aug 2022 at 04:55:32
Issue Comment

Caseycesari

Upgrade Node to 18

Overview

We are currently using Node 16 which is nearing end of life in a year. Node 18 will give us longer to work with.

Forked On 08 Aug 2022 at 04:53:10

Caseycesari

FYI, here is a commit that removed hot reload on another project, for reference.

Commented On 08 Aug 2022 at 04:53:10