Bug 113660 - Wrong label on sidebar slide pane after switching to handout view
Summary: Wrong label on sidebar slide pane after switching to handout view
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Impress (show other bugs)
Version:
(earliest affected)
6.0.0.0.alpha1+
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: difficultyBeginner, easyHack, skillCpp, topicUI
Depends on:
Blocks:
 
Reported: 2017-11-05 16:36 UTC by Tamás Zolnai
Modified: 2018-03-23 21:16 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments
Screenshot of wrong naming (99.71 KB, image/png)
2017-11-05 16:37 UTC, Tamás Zolnai
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Tamás Zolnai 2017-11-05 16:36:22 UTC
Description:
In normal view on the sidebar's Slide pane has a title 'Slide', but when we switch to master slide the pane title become "Master Slide" as expected. The issue here is when I switch to handout view from normal view and then switch to master slides the slide pane page is not set to "Master Slide".

Steps to Reproduce:
1. Open an empty Impress document (we are in normal view)
2. Switch to handout view (View -> Master Handout)
3. Switch to master slide view (View -> Master Slides)

Actual Results:  
Sidepanel's slide pane has a "Slide" title instead of "Master Slide".

Expected Results:
Sidepanel's slide pane should be have a title of "Master Slide".


Reproducible: Always


User Profile Reset: No



Additional Info:


User-Agent: Mozilla/5.0 (Windows NT 6.3; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/61.0.3163.100 Safari/537.36
Comment 1 Tamás Zolnai 2017-11-05 16:37:11 UTC
Created attachment 137539 [details]
Screenshot of wrong naming
Comment 2 Tamás Zolnai 2017-11-05 16:39:04 UTC
Let's turn this into an easy hack. The code is here:
sd/source/ui/sidebar/SlideBackground.cxx

IMPL_LINK(SlideBackground, EventMultiplexerListener, tools::EventMultiplexerEvent&, rEvent, void) method might be a good starting point.
Comment 3 Xisco Faulí 2017-11-09 20:52:24 UTC
Patch in gerrit: https://gerrit.libreoffice.org/#/c/44567/.
Comment 4 Commit Notification 2017-11-17 14:02:35 UTC
Xisco Fauli committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=605b01dc591eaa72788be255baf1f9b9dde76414

tdf#113660: show Master Slide label in handout view

It will be available in 6.0.0.

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.
Comment 5 Commit Notification 2018-03-19 12:11:00 UTC
Xisco Faulí committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=b120932c77332f95ae40d7bd96b972ce87e7693a

tdf#116364: Revert "tdf#113660: show Master Slide label in handout view"

It will be available in 6.1.0.

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.
Comment 6 Xisco Faulí 2018-03-19 12:12:18 UTC
Reopening as the commit has been reverted in https://cgit.freedesktop.org/libreoffice/core/commit/?id=b120932c77332f95ae40d7bd96b972ce87e7693a
It introduced bug 116364 which is more critical...
Comment 7 Commit Notification 2018-03-19 16:10:55 UTC
Xisco Faulí committed a patch related to this issue.
It has been pushed to "libreoffice-6-0":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=855ae519db73b81a64d991b99c8993e6a192df08&h=libreoffice-6-0

tdf#116364: Revert "tdf#113660: show Master Slide label in handout view"

It will be available in 6.0.4.

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.
Comment 8 Commit Notification 2018-03-23 21:15:16 UTC
Xisco Faulí committed a patch related to this issue.
It has been pushed to "libreoffice-6-0-3":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=08ed147729b4b5a4f884b300b0d2e99b6267936a&h=libreoffice-6-0-3

tdf#116364: Revert "tdf#113660: show Master Slide label in handout view"

It will be available in 6.0.3.

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.