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

[2679] Align the input element like the corresponding label #2690

Merged
merged 1 commit into from Dec 13, 2023

Conversation

pcdavid
Copy link
Member

@pcdavid pcdavid commented Nov 29, 2023

Bug: #2679

Capture.video.du.2023-11-29.17-29-01.webm

@pcdavid
Copy link
Member Author

pcdavid commented Nov 29, 2023

It's not ideal for small labels (like the "200" in the video), but better than the previous state where the input was left-aligned and much farther to the actual label's location.

@frouene frouene force-pushed the pcd/fix/direct-edit-input-position branch from 5c695d9 to a3f9876 Compare December 13, 2023 12:42
Bug: #2679
Signed-off-by: Pierre-Charles David <pierre-charles.david@obeo.fr>
Signed-off-by: Florian ROUËNÉ <florian.rouene@obeosoft.com>
@frouene frouene force-pushed the pcd/fix/direct-edit-input-position branch from a3f9876 to c4496d6 Compare December 13, 2023 15:49
@frouene
Copy link
Contributor

frouene commented Dec 13, 2023

The failed test is not related to the PR, I'll merge it

@frouene frouene merged commit 75fc792 into master Dec 13, 2023
3 of 4 checks passed
@frouene frouene deleted the pcd/fix/direct-edit-input-position branch December 13, 2023 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

When triggering a direct edit, the input should be placed at the same location as the label
3 participants