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

added fixture: placeholder#25

Open
ruscoder wants to merge 1 commit into
akatov:masterfrom
ruscoder:master
Open

added fixture: placeholder#25
ruscoder wants to merge 1 commit into
akatov:masterfrom
ruscoder:master

Conversation

@ruscoder

Copy link
Copy Markdown

Use:

@ruscoder ruscoder mentioned this pull request Jun 12, 2014
@akatov

akatov commented Jun 24, 2014

Copy link
Copy Markdown
Owner

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

b1rdex commented Aug 7, 2014

Copy link
Copy Markdown

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

@akatov

akatov commented Aug 7, 2014

Copy link
Copy Markdown
Owner

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