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

TextileParser issues #44

Closed
fbettag opened this issue Aug 10, 2009 · 8 comments
Closed

TextileParser issues #44

fbettag opened this issue Aug 10, 2009 · 8 comments
Labels

Comments

@fbettag
Copy link
Member

fbettag commented Aug 10, 2009

1# * and _. Look at the following:

scala> import net.liftweb.textile._
import net.liftweb.textile._

scala> TextileParser.toHtml("foo bar")
res7: scala.xml.NodeSeq =

foo __bar__

scala> TextileParser.toHtml("bar")
res8: scala.xml.NodeSeq =

bar

scala> TextileParser.toHtml("bar foo")
res9: scala.xml.NodeSeq =

bar *foo*

Something clearly isn't working with __ and ** ;)
#2 links with classes

scala> TextileParser.toHtml(""" "(cmsadmin-link)click here":# """)
res10: scala.xml.NodeSeq =

(cmsadmin-link)click here

Usually this should work too.

@dpp
Copy link
Member

dpp commented Aug 13, 2009

I've fixed issued #1

I don't understand what "this should work too" means. What is the expected output.

@fbettag
Copy link
Member Author

fbettag commented Aug 13, 2009

It should produce a link which has the class cmsadmin-link.

thanks for the quick fix!

@dpp
Copy link
Member

dpp commented Aug 20, 2009

I need to see exactly what HTML you expect to be output... and it would help if you could point to a reference. When I enter the example that you gave into , http://textism.com/tools/textile/index.php I get:

”(cmsadmin-link)click here”:#

@fbettag
Copy link
Member Author

fbettag commented Aug 20, 2009

"(cmsadmin-link)click here":/foo

transforms to:

click here

Done with RedCloth (ruby textile). Syntactically it's just the logical conclusion of the textile specs.

@dpp
Copy link
Member

dpp commented Aug 21, 2009

I've updated Lift to support this construct

@fbettag
Copy link
Member Author

fbettag commented Aug 21, 2009

great! thank you!

@github-importer
Copy link

Imported from GitHub: http://github.com/dpp/liftweb/issues/44/find

@github-importer
Copy link

Imported from Assembla: http://www.assembla.com/spaces/liftweb/tickets/44

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants