Summary: | Gallery: Do not link inserted images | ||
---|---|---|---|
Product: | LibreOffice | Reporter: | Samuel Mehrbrodt <s.mehrbrodt> |
Component: | Libreoffice | Assignee: | Not Assigned <libreoffice-bugs> |
Status: | NEW --- | QA Contact: | |
Severity: | enhancement | ||
Priority: | medium | CC: | barta, jmadero.dev, libreoffice, lohmaier, qubit, rb.henschel, s.mehrbrodt |
Version: | 4.2.0.0.alpha0+ Master | ||
Hardware: | Other | ||
OS: | All | ||
Whiteboard: | target:4.2.0 Confirmed:4.2.0.0.beta1:Ubuntu Confirmed:4.2.0.0.alpha0+:Unknown | ||
i915 platform: | i915 features: | ||
Bug Depends on: | 69079 | ||
Bug Blocks: |
Description
Samuel Mehrbrodt
2013-07-24 20:49:47 UTC
Confirmed. Linking is always "dangerous" and I think any linking to gallery images should be removed. Additionally if a user tries to link to any external resource, he should be warned about the consequences of his action. You are right, it should not be possible to link to gallery images. We discussed this also on the UX-Advise list: http://nabble.documentfoundation.org/Libreoffice-ux-advise-Remove-outdated-Gallery-images-tp4067129p4067444.html I changed the title accordingly. Samuel Mehrbrodt committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=4e3f7d32b80fcb44f20abe634a2035d2c90c9e08 fdo#67274 Always embed images from gallery The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. The mentioned commit fixes this problem only partly. The Problem described in Comment 0 still exists. Can someone else please have a look at it? The Background URL is set here: http://opengrok.libreoffice.org/xref/core/svx/source/gallery2/galbrws2.cxx#372 Then the Image gets inserted here: http://opengrok.libreoffice.org/xref/core/sw/source/ui/shells/basesh.cxx#2662 Somewhere in between the SvxBrushItem (the background) is created as a linked graphic and that must be changed. But I don't know where that happens. Can someone give a hint or take this over? I mark this as MAB because it makes Documents depend on hard-coded image paths. (Adding the Dev list to CC) I'd need some help with the Fix for this bug. Can someone have a look at Comment 5 and help me with this? No idea but I assume code like this: 706 const Graphic aGrf( pGalleryItem->GetGraphic() ); 707 708 if ( nSelType & nsSelectionType::SEL_GRF ) 709 rSh.ReRead( aGrfName, aFltName, &aGrf ); 710 else 711 rSh.Insert( aGrfName, aFltName, aGrf ); Creates that Graphic - and that (somehow) the GraphicObject that gets created from that gets tagged as linked somehow - perhaps chasing rSh.Insert() and re-read to see what parameters they take might be useful (?) CONFIRMED on LO 4.2.0.0.beta1 + Ubuntu 12.04.3 REPRO: - Open LibreOffice - Tools -> Gallery -> Backgrounds - Right-click on aqua (first background in list) and choose: Insert as Background -> Page - Right-click on the changed page background and select 'Page' - Select 'Background' tab - Under the 'File' section, the 'Link' checkbox is checked, and the filepath listed below the 'Browse' button is to a file in 'share/gallery' on disk. EXPECTED: - The background would be embedded in the file. - The 'Link' checkbox would be unchecked Adding repro information to whiteboard not a MAB in my opionion.. * has always been that way for decades, * graphic as page backgrounds are not a very common usecase * easy workaround/manual fix (Goto Format|Page → Background and uncheck [ ] Link) Not saying that this shouldn't be fixed, but that's not so important to have for 4.2.0 (4.2.x is enough IMHO) (This is an automated message.) Setting priority to highest as this is a 4.0 MAB. This is part of an effort to make the importance of MAB reflected in priority too. Moving to mab4.1 (Bug 60270) because: - 4.0 reached EOL (End Of Life) - bug confirmed in later version if issue is still present in 4.2.x current releases please move it to the mab4.2 list (Bug 65675) since 4.1.x is END OF LIFE Removing from MAB - this is an enhancement and does not belong on mab list. Marking as enhancement also. The product works as designed, the request is just to change the design which by definition is an enhancement. |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.