Bug 13844

Summary: WebKitQt Api doesn't provide any way to append html code to the previous code
Product: WebKit Reporter: sebastien le faou <plicplicplic>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED INVALID    
Severity: Normal CC: hausmann, manyoso
Priority: P2    
Version: 523.x (Safari 3)   
Hardware: Other   
OS: Linux   
Attachments:
Description Flags
a solution to this bug mjs: review-

sebastien le faou
Reported 2007-05-23 09:06:05 PDT
Description: If I have an application, like a chat, which uses WebKitQt to render the messages, there are no possibilities to append the new html message codes to the former one. Indeed, I have to load the entire html code, including the previous messages and the new one. A function which allow to add new html codes without removing the old one could be a good thing to have
Attachments
a solution to this bug (1.15 KB, patch)
2007-05-23 09:22 PDT, sebastien le faou
mjs: review-
sebastien le faou
Comment 1 2007-05-23 09:22:45 PDT
Created attachment 14688 [details] a solution to this bug I created 2 functions void openStringAppend(const QString &htmlString) This function is rendering the html code from a String and it also append this html code to the previous html (The one which was rendered before) void closeStringAppend() This function is supposed to finish the display if the previous function (openStringAppend(...)) is used
Simon Hausmann
Comment 2 2007-05-29 00:32:39 PDT
I admit I actually liked the KHTML API with regards to this feature. How about void beginHtml(const QUrl &baseUrl = QUrl()); void appendHtml(const QByteArray &html); void endHtml();
Maciej Stachowiak
Comment 3 2007-05-29 00:57:18 PDT
Once WebKitQt supports a DOM API, you could do this via the Document interface with the open/write/close methods. I think this implementation is not as safe as those with regards to maintaining internal state properly. Even if you want to add it this way, I'd suggest calling open/write/close on the document as the implementation instead of using begin() / end().
Maciej Stachowiak
Comment 4 2007-05-29 01:52:22 PDT
Comment on attachment 14688 [details] a solution to this bug r- for wrong implementation and to consider API feedback
Adam Treat
Comment 5 2009-07-14 18:05:37 PDT
This has been fixed with the new DOM API.
Note You need to log in before you can comment on or make changes to this bug.