Skip to content
This repository has been archived by the owner on Jan 7, 2020. It is now read-only.

added fixture: placeholder #25

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

added fixture: placeholder #25

wants to merge 1 commit into from

Conversation

ruscoder
Copy link

Use:

@ruscoder ruscoder mentioned this pull request Jun 12, 2014
@akatov
Copy link
Owner

akatov commented Jun 24, 2014

hmm, not sure it makes sense to have a simple placeholder like this.
Given this is a full contenteditable div with all sorts of css stylings possible I might want something similar for a placeholder. Not sure what the best way to do it would be though...

@b1rdex
Copy link

b1rdex commented Aug 7, 2014

@akatov here is the best way: http://stackoverflow.com/a/18368720

@akatov
Copy link
Owner

akatov commented Aug 7, 2014

I meant not just CSS stylings, I also meant the inclusion of arbitrary html inside the placeholder which doesn't seem to be possible in the solution from stackoverflow, as < and > get escaped. It may work in @ruscoder's version, however, I won't be merging this without any test fixtures!
In any case, I wonder whether it is better to bind the placeholder value to another model instead of the contents of an attribute value. Thoughts?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants