| Summary: | Bookmarks created by drag and drop: 'Name' should be the page title, not the URL | ||
|---|---|---|---|
| Product: | [Applications] keditbookmarks | Reporter: | tmb-kde-bugs |
| Component: | general | Assignee: | Konqueror Bugs <konqueror-bugs-null> |
| Status: | CONFIRMED --- | ||
| Severity: | wishlist | CC: | aristsakas, frank78ac, pedrogomes81, thomas.bettler, tpm01 |
| Priority: | NOR | ||
| Version First Reported In: | 1.0 | ||
| Target Milestone: | --- | ||
| Platform: | Debian testing | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
|
Description
tmb-kde-bugs
2001-05-27 13:38:03 UTC
*** Bug 36148 has been marked as a duplicate of this bug. *** this is really a feature, but hopefully i'll get it finished by 3.2 in any case. Alex *** Bug 119199 has been marked as a duplicate of this bug. *** I think fixing this is not as easy as duplicating the menu's "Add Bookmark" functionality (as some commenters in the duplicate reports suggested). "Add Bookmark" happens inside the application while drag&drop is a cross-application thing (note that you can drag the favicons from Konqueror's location bar or the tabs to the Firefox bookmark toolbar and vice versa). To fix this issue, 2 things must be changed: 1. When the drag event is started, the QDrag object must be given MIME data not only for the URL, but also for the page title. Firefox seems to provide this in the MIME types "text/x-moz-url" and "_NETSCAPE_URL" (both contain first the URL, and then the page title). In Konqueror, the creation of the QDrag object seems to happen in KonqDraggableLabel::mouseMoveEvent(). 2. When the drop event is accepted, the page title must be read from the MIME data of the drop event. Both KEditBookmarks and the Konqueror bookmarks bar parse the MIME data using KBookmark::List::fromMimeData(). Changing this function accordingly would therefore fix this wish and all duplicates. The problem is that both KonqDraggableLabel::mouseMoveEvent() and KBookmark::List::fromMimeData() rely on KUrl's methods for the MIME data generation and parsing, and KUrl does not seem to support a title in addition to the actual URL. So the required code change appears to be non-trivial :-( *** Bug 190458 has been marked as a duplicate of this bug. *** |