Issue with Entry's Placeholder

[antonmaju]

Issue:
If entry has both placeholder and text value, it will still render both text and placeholder initially.
This causes text to be hard to read.

Expected:
Entry won't render placeholder if text is present.

Device Info

• OS: Tested on Windows 10 & Ubuntu 18.04
• Go version: 1.14
• Fyne version: 1.3.2

[andydotxyz]

Thanks for the report, not sure how this slipped in.
Does it fix if you resize, or only if you type something?

[antonmaju]

Hi @andydotxyz,
Performing resize doesn't fix the issue, but getting focus or calling FocusGained() manually fixes it

[andydotxyz]

For some reason I am struggling to replicate this sorry - can you share the code that exhibits that result please?

[antonmaju]

Hi @andydotxyz

The example is like this

import (
	"fyne.io/fyne"
	"fyne.io/fyne/app"
	"fyne.io/fyne/layout"
	"fyne.io/fyne/widget"
)

func TextBoxTest() {
	wApp := app.New()
	w := wApp.NewWindow("Fyne Test")
	w.Resize(fyne.Size{ Width: 400, Height: 300})
	w.SetFixedSize(true)
	lblTitle := widget.NewLabel("Title")
	txtTitle := widget.NewEntry()
	txtTitle.Text = "Test"
	txtTitle.SetPlaceHolder("Default text")
	//txtTitle.FocusGained()
	content := fyne.NewContainerWithLayout(layout.NewVBoxLayout(), lblTitle, txtTitle)
	w.SetContent(content)
	w.ShowAndRun()
}

Thanks !

[andydotxyz]

Oh thanks so much - our test utility was somehow not triggering this bug but now I can replicate/

[antonmaju]

HI @andydotxyz

Sorry I just realized, if I use SetText method instead of Text property directly it works.

[andydotxyz]

Not a problem, that's a fix on develop now if you'd like to test :)

[Conan520]

I also had this problem, but I fixed it with SetText(""), I don't know why