Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Lidar Blog Post #2916

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

feat: Lidar Blog Post #2916

wants to merge 13 commits into from

Conversation

github-actions[bot]
Copy link
Contributor

This PR was automatically created by the "Issue opened (new blog)" workflow.

closes #2915

Copy link

netlify bot commented Dec 13, 2024

Deploy Preview for gis-utah-gov ready!

Name Link
🔨 Latest commit 6aa91c1
🔍 Latest deploy log https://app.netlify.com/sites/gis-utah-gov/deploys/6765bf04c0f1000008635566
😎 Deploy Preview https://deploy-preview-2916--gis-utah-gov.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 97 (🟢 up 1 from production)
Accessibility: 98 (🔴 down 2 from production)
Best Practices: 100 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports

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

@miriamseely miriamseely requested a review from rkelson December 13, 2024 18:00
@miriamseely miriamseely changed the title feat: create new blog post feat: Lidar Blog Post Dec 13, 2024
Copy link
Member

@steveoh steveoh left a comment

Choose a reason for hiding this comment

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

looks good!

The lazy eager image stuff allows the page to load faster if the image is expected to not be visible on the initial page load.

Here's an article about writing alt text for images that just came through my feed I found very helpful so I thought I'd share as it's not very intuitive.

https://www.nngroup.com/articles/alt-text-usability/

@rkelson rkelson marked this pull request as ready for review December 18, 2024 14:48
@rkelson
Copy link
Member

rkelson commented Dec 18, 2024

how do we get this to deploy

@steveoh
Copy link
Member

steveoh commented Dec 18, 2024

how do we get this to deploy

You can click on the failed deploy log to see the errors. This error shows something wrong with the cover image.

image

Turns out it was the wrong path and had a quote

a couple misc. edits

I think this looking great!

The only problem I have is we have covered as much of the state as we can but really do not have "statewide coverage" considering the hole over dugway proving grounds. The hole is in the graphic on page https://gis.utah.gov/products/sgid/elevation/lidar/#2022-western--eastern-utah
@miriamseely can you make some adjustments to accommodate this fact?
@steveoh
Copy link
Member

steveoh commented Dec 18, 2024

moving ricks comments out of the commit for easier visibility

a couple misc. edits

I think this looking great!

The only problem I have is we have covered as much of the state as we can but really do not have "statewide coverage" considering the hole over dugway proving grounds. The hole is in the graphic on page https://gis.utah.gov/products/sgid/elevation/lidar/#2022-western--eastern-utah
@miriamseely can you make some adjustments to accommodate this fact?

@miriamseely
Copy link
Collaborator

@steveoh @rkelson Happy Friday, I made some changes to the post to reflect that we don't have statewide coverage as of yet, fixed the date, and made some other minor adjustments. If the post looks good to you both, then I think we're ready to publish. Thank you for all your help with this one!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✍️ Lidar Blog Post
3 participants