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

Fix: Allow editing of long strings in the PCodeInput component #871

Merged
merged 5 commits into from
Aug 18, 2023

Conversation

zhen0
Copy link
Member

@zhen0 zhen0 commented Aug 1, 2023

We've had reports of a bug in our code input. The video before shows the issue - I can't click into a long string input:
https://github.com/PrefectHQ/prefect-design/assets/40272060/d4c4a2a8-79ac-4abf-af23-ced7ac1a2369

This PR refactors how we set the width of the textarea in the PCodeInput to set it according to the width of the input/the width of the sibling view-container.

Screen.Recording.2023-08-01.at.3.10.04.PM.mov

@netlify
Copy link

netlify bot commented Aug 1, 2023

Deploy Preview for prefect-design ready!

Name Link
🔨 Latest commit 020740f
🔍 Latest deploy log https://app.netlify.com/sites/prefect-design/deploys/64dfbfc1daec75000893ce0d
😎 Deploy Preview https://deploy-preview-871--prefect-design.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.

@zhen0 zhen0 changed the title Code width Fix: Allow editing of long strings in the PCodeInput component Aug 1, 2023
@zhen0 zhen0 marked this pull request as ready for review August 18, 2023 18:38
src/components/CodeInput/PCodeInput.vue Outdated Show resolved Hide resolved
Co-authored-by: pleek91 <craig@prefect.io>
@zhen0 zhen0 merged commit f4549bc into main Aug 18, 2023
6 checks passed
@zhen0 zhen0 deleted the code-width branch August 18, 2023 19:35
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.

3 participants