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

Feature/pxweb2 90 label #38

Merged
merged 4 commits into from
Feb 13, 2024
Merged

Feature/pxweb2 90 label #38

merged 4 commits into from
Feb 13, 2024

Conversation

MikaelNordberg
Copy link
Contributor

Label component

Copy link

netlify bot commented Feb 12, 2024

Deploy Preview for pxweb2 ready!

Name Link
🔨 Latest commit 5511039
🔍 Latest deploy log https://app.netlify.com/sites/pxweb2/deploys/65cb3bd7dc119d00080aefdb
😎 Deploy Preview https://deploy-preview-38--pxweb2.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 configuration.

@PerIngeVaaje PerIngeVaaje self-requested a review February 12, 2024 11:43
@PerIngeVaaje PerIngeVaaje marked this pull request as ready for review February 12, 2024 11:46
...rest
}: LabelProps) {
return (
<p
Copy link
Contributor

Choose a reason for hiding this comment

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

This should probably be a span. A <p> tag is for paragraphs

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@michaelpande

Now is a label element with the possibility to set htmlFor:

  <Label htmlFor="fname">First name:</Label><br />
  <input type="text" id="fname" name="fname" defaultValue="John" /><br />

KentMossback
KentMossback previously approved these changes Feb 13, 2024
Copy link
Contributor

@KentMossback KentMossback 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

Copy link
Contributor

@KentMossback KentMossback left a comment

Choose a reason for hiding this comment

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

Still looks good

Copy link
Contributor

@PerIngeVaaje PerIngeVaaje left a comment

Choose a reason for hiding this comment

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

Looks OK

@KentMossback KentMossback merged commit 3e9a0fa into main Feb 13, 2024
8 checks passed
@KentMossback KentMossback deleted the feature/PXWEB2-90-Label branch February 13, 2024 09:56
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.

4 participants