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

Update the way text field is displayed #272

Closed
wants to merge 2 commits into from
Closed

Update the way text field is displayed #272

wants to merge 2 commits into from

Conversation

rafallus
Copy link

Fixed text field displaying the "mouse over" state when already focused.
This causes problems as field is not going to show the focused state until you move the mouse out of the field

Fixed text field displaying the "mouse over" state when already focused. 
This causes problems as field is not going to show the focused state until you move the mouse out of the field
Updated way text field is shown
@rafallus rafallus changed the title Updated the way text field is displayed Update the way text field is displayed Oct 21, 2017
@kotcrab
Copy link
Owner

kotcrab commented Oct 29, 2017

The current way is the intended behaviour.
If anything I would change it to always show backgroundOver when field has keyboard focus by modifying that if to:

if (!disabled && style.backgroundOver != null && (clickListener.isOver() || focused)) {
	background = style.backgroundOver;
}

@kotcrab
Copy link
Owner

kotcrab commented Nov 6, 2017

I pushed the change from my previous comment to always show backgroundOver when text field has keyboard focus.

@rafallus
Copy link
Author

rafallus commented Nov 6, 2017 via email

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.

2 participants