deyz112 / fauxbar

Automatically exported from code.google.com/p/fauxbar
0 stars 0 forks source link

Bookmarks List in New Tab Page Doesn't Sanitize Bookmark URLs Properly #124

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Create a bookmark on the toolbar with this url: 
javascript:"<title>hi</title><h1>hi</h1>"
2. Open the bookmarks menu on the Fauxbar page.

What is the expected output? What do you see instead?
See attachment.

Which version of Fauxbar and Chrome are you using, and which operating
system?
Chrome 21.0.1180.60, Fauxbar Lite 1.2.10, Win7.

Please provide any additional information below:
It also renders nameless bookmarks poorly.

Original issue reported on code.google.com by DeathByN...@gmail.com on 7 Aug 2012 at 8:33

Attachments:

GoogleCodeExporter commented 8 years ago
Aye, URLs starting with javascript: need to be handled better, for both the 
toolbar and Address Box. Errors and bad formatting abound.

Untitled bookmarks need some love, too - a simple   should solve the height 
issues.

Original comment by fauxbar....@gmail.com on 8 Aug 2012 at 11:47

GoogleCodeExporter commented 8 years ago

Original comment by fauxbar....@gmail.com on 18 Apr 2013 at 4:58

GoogleCodeExporter commented 8 years ago
This has now been fixed in Fauxbar v1.2.11.

Original comment by fauxbar....@gmail.com on 18 Apr 2013 at 6:47