Bug 125531 - Empty CMIS properties dialog
Summary: Empty CMIS properties dialog
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: UI (show other bugs)
Version:
(earliest affected)
6.3.0.0.alpha1+
Hardware: All All
: medium enhancement
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: difficultyBeginner, easyHack, skillCpp, topicUI
Depends on:
Blocks: CMIS Dialog
  Show dependency treegraph
 
Reported: 2019-05-27 20:11 UTC by Thorsten Wagner
Modified: 2019-06-02 15:40 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments
Screenshot (33.91 KB, image/png)
2019-05-27 20:11 UTC, Thorsten Wagner
Details
Screenshot of CMIS dialog with right-handed scrollbar (98.68 KB, image/png)
2019-06-01 10:34 UTC, Thorsten Wagner
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Thorsten Wagner 2019-05-27 20:11:05 UTC
Description:
Without CMIS properties related dialog within document properties is empty. Only a grabled scrollbar is shown (see screenshot).

Steps to Reproduce:
(1) Open any LO application, e.g. Writer

(2) Open File / Properties dialog and select CMIS properties tab

Actual Results:
An empty dialog with a grabled scrollbar is shown

Expected Results:
A message indicating no CMIS properties is shown, scrollbar is not shown or shown at full height


Reproducible: Always


User Profile Reset: No


OpenGL enabled: Yes

Additional Info:
Comment 1 Thorsten Wagner 2019-05-27 20:11:27 UTC
Created attachment 151712 [details]
Screenshot
Comment 2 Julien Nabet 2019-05-27 20:18:19 UTC
On pc Debian x86-64 with master sources updated today, I could reproduce this.
=> I put it at NEW.

Xisco/Heiko: is it a bug, an enhancement, ...? Thought it may interest UX people.
Comment 3 Heiko Tietze 2019-05-28 10:58:39 UTC
I'd say enhancement. What bothers me is the small object at top left. Best solution is IMHO to hide CMIS when the document is not stored remotely.
Comment 4 Julien Nabet 2019-05-28 11:45:37 UTC
Code pointer:
https://opengrok.libreoffice.org/xref/core/sfx2/source/dialog/dinfdlg.cxx?r=da92911d#1145

I tried this:
diff --git a/sfx2/source/dialog/dinfdlg.cxx b/sfx2/source/dialog/dinfdlg.cxx
index ec3f57656ab9..87d1adbb1c38 100644
--- a/sfx2/source/dialog/dinfdlg.cxx
+++ b/sfx2/source/dialog/dinfdlg.cxx
@@ -1142,7 +1142,8 @@ SfxDocumentInfoDialog::SfxDocumentInfoDialog( vcl::Window* pParent,
     m_nDocInfoId = AddTabPage("general", SfxDocumentPage::Create);
     AddTabPage("description", SfxDocumentDescPage::Create);
     AddTabPage("customprops", SfxCustomPropertiesPage::Create);
-    AddTabPage("cmisprops", SfxCmisPropertiesPage::Create);
+    if (rInfoItem.GetCmisProperties().hasElements())
+        AddTabPage("cmisprops", SfxCmisPropertiesPage::Create);
     AddTabPage("security", SfxSecurityPage::Create);
 }

+ remove assert here:
diff --git a/sfx2/source/dialog/tabdlg.cxx b/sfx2/source/dialog/tabdlg.cxx
index 79e2ad750ada..bc349c8d9c13 100644
--- a/sfx2/source/dialog/tabdlg.cxx
+++ b/sfx2/source/dialog/tabdlg.cxx
@@ -562,8 +562,6 @@ bool SfxTabDialog::StartExecuteAsync( VclAbstractDialog::AsyncContext &rCtx )
 
 void SfxTabDialog::Start_Impl()
 {
-    assert(m_pImpl->aData.size() == m_pTabCtrl->GetPageCount()
-            && "not all pages registered");
     sal_uInt16 nActPage = m_pTabCtrl->GetPageId( 0 );
 
     // load old settings, when exists

It doesn't work. Cmis tab still appears but you can't select it.
Comment 5 Thorsten Wagner 2019-05-30 22:48:02 UTC
Scroll bar artefact has been replaced by a right handed full size scroll bar. Setting status to resolved / works for me.
Comment 6 Heiko Tietze 2019-06-01 07:39:01 UTC
(In reply to Thorsten Wagner from comment #5)
> Scroll bar artefact has been replaced by a right handed full size scroll
> bar. Setting status to resolved / works for me.

You likely want to resolve bug 101443 and not the CMIS issue. :-)
Comment 7 Thorsten Wagner 2019-06-01 10:33:20 UTC
No, behaviour of CMIS dialog has changed (see screenshot). Please set issue to resolved if this is the intended behaviour.
Comment 8 Thorsten Wagner 2019-06-01 10:34:13 UTC
Created attachment 151825 [details]
Screenshot of CMIS dialog with right-handed scrollbar
Comment 9 Heiko Tietze 2019-06-01 11:19:56 UTC
The scrollbar might be a bit better but it is still pointless to show the tab. And for me the dialog has a tiny but focused control at top left. Better we hide the tab completely.
Comment 10 Thorsten Wagner 2019-06-01 22:38:44 UTC
I totally agree. Furthermore the order of buttons within dialog boxes has changed (see screenshots). Is this behaviour intended?