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

BF: Fix to textbox colors #3222

Merged
merged 4 commits into from
Oct 22, 2020
Merged

Conversation

TEParsons
Copy link
Contributor

Uses Color class as a one-off fix to be over-written when the class is fully implemented

Todd added 3 commits October 16, 2020 13:30
@coveralls
Copy link

coveralls commented Oct 16, 2020

Coverage Status

Coverage decreased (-0.004%) to 46.477% when pulling 8c33b43 on TEParsons:textboxColorPatchup into 286b409 on psychopy:master.

@TEParsons TEParsons closed this Oct 16, 2020
@TEParsons TEParsons reopened this Oct 16, 2020
@TEParsons
Copy link
Contributor Author

@peircej so there was already code in there to draw the caret even when no character vertices are available, it was just calculating default vertices wrong so the width became 0. The position isn't quite right still, but it's close enough for now as it'll all change when the Size/Position classes are in anyway.

@peircej peircej merged commit 40628c3 into psychopy:master Oct 22, 2020
peircej pushed a commit that referenced this pull request Oct 22, 2020
* BF: Use Color class to handle textbox pallette, bypasses problems with colors being innaccurate in some spaces

* Revert "BF: Use Color class to handle textbox pallette, bypasses problems with colors being innaccurate in some spaces"

This reverts commit 193ed81

* BF: Remove Textbox color change on selection, will reinstate when Color class is in use

* BF: Caret now appears when textbox is empty

Co-authored-by: Todd <TEParsons>
@TEParsons TEParsons deleted the textboxColorPatchup branch February 11, 2022 09:54
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