Set pointer-events-auto class on HomepageHero text

[AdamAnSubtractM]

Set pointer-events to none for the HomepageHero canvas so that text can be selected.

[CLAassistant]

All committers have signed the CLA.

[lucacasonato]

Not allowing to select the hero text was intentional: #115. Is there a specific issue you are running into?

[AdamAnSubtractM]

Not allowing to select the hero text was intentional: #115. Is there a specific issue you are running into?

Did not realize this was intentional. I was trying to copy and paste the text to send to someone and noticed I could not. Seemed like a usability issue to me and opened a PR. If the JSR team feels users shouldn't be able to select text on the page, then I don't think Its my place to say otherwise.

Let me know if you want me to close this, otherwise, feel free to decline. Thanks for your time.

[josh-collinsworth]

@AdamAnSubtractM Thanks for the contribution! I agree; this is an issue for usability.

However, this approach prevents some interactions with the background. I believe the better way to achieve the desired outcome would be to simply add the pointer-events-auto class to the <div> that contains the text in question. That will allow its text to be selectable, while keeping everything around it as-is.

[AdamAnSubtractM]

@AdamAnSubtractM Thanks for the contribution! I agree; this is an issue for usability.

However, this approach prevents some interactions with the background. I believe the better way to achieve the desired outcome would be to simply add the pointer-events-auto class to the <div> that contains the text in question. That will allow its text to be selectable, while keeping everything around it as-is.

Thanks @josh-collinsworth . I'll update the PR with your suggestions.

[AdamAnSubtractM]

@AdamAnSubtractM Thanks for the contribution! I agree; this is an issue for usability.

However, this approach prevents some interactions with the background. I believe the better way to achieve the desired outcome would be to simply add the pointer-events-auto class to the <div> that contains the text in question. That will allow its text to be selectable, while keeping everything around it as-is.

@josh-collinsworth - PR updated, thanks.

[josh-collinsworth]

Perfect, thanks!