That's the result of my patch. I wanted something like a list (with dots), but unfortunately using
<ul>
<li>item</li>
<li>item</li>
</ul>
produce a bad looking space between adiacent list items, even if there's no <br> between them.
So i tried to manually put a dot ( "•" ) to resemble a list, but i see that neither this works... i can try to break any long path string so you won't get that thing... or remove completely that dot :)
EDIT: Or we can use <li></li> without a <ul> element, that should work... let me try..
Ok, resolved, it was just a matter of a \n that became a <br> for no reason.
(http://img211.imageshack.us/img211/8027/before2bu.th.jpg) (http://img211.imageshack.us/my.php?image=before2bu.jpg)
(http://img56.imageshack.us/img56/7195/after7ny.th.jpg) (http://img56.imageshack.us/my.php?image=after7ny.jpg)
Now you see why i wanted a list...
here's my new patch:
Index: src/src/main.cpp
===================================================================
--- src/src/main.cpp (revision 2573)
+++ src/src/main.cpp (working copy)
@@ -1449,27 +1449,31 @@
links << _T("<b>Recent projects</b><br>\n");
if (m_ProjectsHistory.GetCount())
{
+ links << _T("<ul>");
for (int i = 0; i < 9; ++i)
{
if (i >= (int)m_ProjectsHistory.GetCount())
break;
- links << wxString::Format(_T("• <a href=\"CB_CMD_OPEN_HISTORY_PROJECT_%d\">%s</a><br>\n"),
+ links << wxString::Format(_T("<li><a href=\"CB_CMD_OPEN_HISTORY_PROJECT_%d\">%s</a></li>"),
i + 1, m_ProjectsHistory.GetHistoryFile(i).c_str());
}
+ links << _T("</ul><br>");
}
else
links << _T(" No recent projects<br>\n");
-
+
links << _T("<br><b>Recent files</b><br>\n");
if (m_FilesHistory.GetCount())
{
+ links << _T("<ul>");
for (int i = 0; i < 9; ++i)
{
if (i >= (int)m_FilesHistory.GetCount())
break;
- links << wxString::Format(_T("• <a href=\"CB_CMD_OPEN_HISTORY_FILE_%d\">%s</a><br>\n"),
+ links << wxString::Format(_T("<li><a href=\"CB_CMD_OPEN_HISTORY_FILE_%d\">%s</a></li>"),
i + 1, m_FilesHistory.GetHistoryFile(i).c_str());
}
+ links << _T("</ul>");
}
else
links << _T(" No recent files<br>\n");