Ben Fritz wrote:

> On Jul 14, 6:22 am, Bram Moolenaar <[email protected]> wrote:
>>
>> I haven't tried it, mostly looks OK.  Except:
>>
>> -" vim: noet sw=2 sts=2
>> +" Make sure any patches will probably use consistent indent
>> +"   vim: ts=2 sw=2 sts=2 et
>>
>> All runtime files use a tabstop setting of 8.
>>
>> About these deleted lines:
>>
>>         -" hover opening implies dynamic folding
>>         -if exists("s:html_hover_unfold")
>>         -  let s:html_dynamic_folds = 1
>>         -endif
>>
>> When someone sources 2html.vim directly, not using :TOhtml, won't
>> something break now?
>>
>
> Bram just pushed a patch from me to Mercurial that fixes these and
> also adds a progress bar. Please try it out and send feedback.
>
> I'm especially interested if the diff view still looks nice on
> Internet Explorer 6 (or earlier...but I'm not really interested in
> supporting something that ancient unless it's an easy fix). And of
> course, trying it out with various combinations of options. I've
> tested the script output (with html_use_css set) on Firefox, Chrome,
> Safari, Opera, IE8, and IE7 on Windows and so far, so good, as far as
> I can tell.
>
> I know the javascript for dynamic folding in diff view is broken for
> more than 2 files (hard-coded 2 instead of a call to len() as I
> intended). I'll get an update out for that soon. In the meantime, if
> you NEED this feature, it should be easy to fix in autoload/
> tohtml.vim. Bram also saw that the syntax highlighting on diff lines
> is lost. This does not look like a regression (at least not since just
> before diff view was added, it has to do with the way we get the ID to
> highlight with) but I'm going to try to fix that as well.

Hi Ben

I just tried ":TOhtml" with Vim-7.3a (2322:1a6c5a3ff687).
The progress bar looks neat.

I see a bug: part of CSS style shows up in the rendered HTML
at the bottom of the page.

Steps to reproduce:

1) use the following ~/.vimrc

set nocompatible
syntax on
filetype on
filetype plugin on
let html_use_css=1

2) generate a HTML file:

$ vim vim/src/popupmnu.c -c 'TOhtml' -c xa

3) Open generated vim/src/popupmnu.c.html in Firefox
and observe at the bottom of the page that the following
line is rendered (it should not be rendered):

.Statement { color: #ffff00; }

The bottom of the generated HTML source looks like this:

$ tail  vim/src/popupmnu.c.html
    <span class="Type">int</span>
pum_get_height()
{
    <span class="Statement">return</span> pum_height;
}
.Statement { color: #ffff00; }
<span class="PreProc">#endif</span>
</pre>
</body>
</html>

The line ".Statement CSS {...}" should be in the HTML header,
not in the body.

Regards
-- Dominique

-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

Raspunde prin e-mail lui