[vlmc-devel] [PATCH] Make toolbar unmovable
Paweł Goliński
golpaw1 at gmail.com
Mon Mar 7 13:45:23 CET 2016
Yes, the toolbar is the only widget affected. I can move it however if I disable all
widget initialization in MainWindow.cpp, but I haven’t looked much beyond that yet.
AFAIC, only I was able to reproduce this issue, from the stack trace it seems OS X specific.
> Wiadomość napisana przez Hugo Beauzée-Luyssen <hugo at beauzee.fr> w dniu 07.03.2016, o godz. 13:40:
>
> On 03/07/2016 10:51 AM, Paweł Goliński wrote:
>> Here is how to reproduce it:
>> https://www.youtube.com/watch?v=fhvy2AehYQ8&feature=em-upload_owner
>>
>> I know this doesn’t make a good fix, but it’s a workaround (do the users
>> really move the toolbar that
>> frequently? Either way if they move it, they’ve just lost their entire
>> work) until this is properly fixed.
>>
>> Here’s the stacktrace: http://pastebin.com/Sr3wDw47 (I don’t know if the
>> crashhandler was disabled, but in CMakeLists.txt there is
>> this fragment of code:
>> "IF(NOT APPLE)
>> SET(WITH_CRASHHANDLER_GUI TRUE CACHE BOOL "Enable the crash handler
>> GUI (with backtrace and restart capabilities)")
>> SET(WITH_CRASHHANDLER TRUE CACHE BOOL "Enable the crash handler")
>> ELSE(NOT APPLE)
>> #FIXME: Mac OS port does n't start if crash handler is enabled.
>> SET(WITH_CRASHHANDLER_GUI FALSE CACHE BOOL "Disable crash handler
>> gui...")
>> SET(WITH_CRASHHANDLER False CACHE BOOL "Disable the crash handler")
>> #SET(CMAKE_OSX_ARCHITECTURES i386)
>> #SET(CMAKE_INSTALL_PREFIX "${CMAKE_CURRENT_BINARY_DIR}")
>> ENDIF(NOT APPLE)”
>> That suggests it is defaultly disabled…)
>>
>>> Wiadomość napisana przez Hugo Beauzée-Luyssen <hugo at beauzee.fr
>>> <mailto:hugo at beauzee.fr>> w dniu 07.03.2016, o godz. 09:57:
>>>
>>> On 03/07/2016 03:21 AM, Paweł Goliński wrote:
>>>> When user tried to move toolbar, VLMC crashed with
>>>> segmentation fault. This patch made toolbar unmovable,
>>>> which solves the problem.
>>>> ---
>>>> src/Gui/ui/MainWindow.ui | 31 ++++++++++++++++++++-----------
>>>> 1 file changed, 20 insertions(+), 11 deletions(-)
>>>>
>>>> diff --git a/src/Gui/ui/MainWindow.ui b/src/Gui/ui/MainWindow.ui
>>>> index 4640741..c8f1465 100644
>>>> --- a/src/Gui/ui/MainWindow.ui
>>>> +++ b/src/Gui/ui/MainWindow.ui
>>>> @@ -27,7 +27,7 @@
>>>> <x>0</x>
>>>> <y>0</y>
>>>> <width>800</width>
>>>> - <height>17</height>
>>>> + <height>22</height>
>>>> </rect>
>>>> </property>
>>>> <widget class="QMenu" name="menuEdit">
>>>> @@ -99,6 +99,15 @@
>>>> <property name="windowTitle">
>>>> <string>VLMC Toolbar</string>
>>>> </property>
>>>> + <property name="movable">
>>>> + <bool>false</bool>
>>>> + </property>
>>>> + <property name="allowedAreas">
>>>> + <set>Qt::TopToolBarArea</set>
>>>> + </property>
>>>> + <property name="floatable">
>>>> + <bool>false</bool>
>>>> + </property>
>>>> <attribute name="toolBarArea">
>>>> <enum>TopToolBarArea</enum>
>>>> </attribute>
>>>> @@ -151,7 +160,7 @@
>>>> </property>
>>>> <property name="statusTip">
>>>> <string>Project Preferences</string>
>>>> - </property>
>>>> + </property>
>>>> </action>
>>>> <action name="actionPreview">
>>>> <property name="checkable">
>>>> @@ -189,7 +198,7 @@
>>>> </property>
>>>> <property name="statusTip">
>>>> <string>Makes VLMC Fullscreen</string>
>>>> - </property>
>>>> + </property>
>>>> </action>
>>>> <action name="actionRender">
>>>> <property name="icon">
>>>> @@ -216,7 +225,7 @@
>>>> </property>
>>>> <property name="statusTip">
>>>> <string>Loads a VLMC Project</string>
>>>> - </property>
>>>> + </property>
>>>> </action>
>>>> <action name="actionSave">
>>>> <property name="icon">
>>>> @@ -228,7 +237,7 @@
>>>> </property>
>>>> <property name="statusTip">
>>>> <string>Saves a VLMC Project</string>
>>>> - </property>
>>>> + </property>
>>>> </action>
>>>> <action name="actionImport">
>>>> <property name="icon">
>>>> @@ -252,7 +261,7 @@
>>>> </property>
>>>> <property name="statusTip">
>>>> <string>Project Preferences</string>
>>>> - </property>
>>>> + </property>
>>>> </action>
>>>> <action name="actionSave_As">
>>>> <property name="icon">
>>>> @@ -264,7 +273,7 @@
>>>> </property>
>>>> <property name="statusTip">
>>>> <string>Save a VLMC project as a different name</string>
>>>> - </property>
>>>> + </property>
>>>> </action>
>>>> <action name="actionClose_Project">
>>>> <property name="icon">
>>>> @@ -276,7 +285,7 @@
>>>> </property>
>>>> <property name="statusTip">
>>>> <string>Closes a VLMC project</string>
>>>> - </property>
>>>> + </property>
>>>> </action>
>>>> <action name="actionCrash">
>>>> <property name="text">
>>>> @@ -293,7 +302,7 @@
>>>> </property>
>>>> <property name="statusTip">
>>>> <string>Undoes the previous action</string>
>>>> - </property>
>>>> + </property>
>>>> </action>
>>>> <action name="actionRedo">
>>>> <property name="icon">
>>>> @@ -305,7 +314,7 @@
>>>> </property>
>>>> <property name="statusTip">
>>>> <string>Redoes the previous action</string>
>>>> - </property>
>>>> + </property>
>>>> </action>
>>>> <action name="actionShare_On_Internet">
>>>> <property name="icon">
>>>> @@ -362,7 +371,7 @@
>>>> </property>
>>>> <property name="statusTip">
>>>> <string>Use the Scissor Tool to cut regions in the timeline</string>
>>>> - </property>
>>>> + </property>
>>>> </action>
>>>> </widget>
>>>> <resources>
>>>>
>>>
>>> Hi,
>>>
>>> I can't reproduce the issue, but this would not be the proper fix
>>> anyway, unless the crash lies in Qt for some reason.
>>> Do you have a backtrace?
>>>
>>> (Remember that to get a usable backtrace, VLMC needs to be built
>>> without the crash handler)
>>>
>>> Regards,
>>> _______________________________________________
>>> Vlmc-devel mailing list
>>> Vlmc-devel at videolan.org <mailto:Vlmc-devel at videolan.org>
>>> https://mailman.videolan.org/listinfo/vlmc-devel
>>
>>
>>
>> _______________________________________________
>> Vlmc-devel mailing list
>> Vlmc-devel at videolan.org
>> https://mailman.videolan.org/listinfo/vlmc-devel
>>
>
> This definitely looks like a platform specific issue to me... could you check if some other people can reproduce a similar issue with docked widgets?
>
> Is the toolbar the only widget that makes the application crash when you move it?
>
> Regards,
> _______________________________________________
> Vlmc-devel mailing list
> Vlmc-devel at videolan.org
> https://mailman.videolan.org/listinfo/vlmc-devel
More information about the Vlmc-devel
mailing list