Differential D10844 Diff 28248 lib/documentview/documentviewcontroller.cpp

Changeset View

Standalone View

View Options

lib/documentview/documentviewcontroller.cpp

Show First 20 Lines • Show All 73 Lines • ▼ Show 20 Line(s)73{
74 DocumentViewController* q; 74​ DocumentViewController* q;
75 KActionCollection* mActionCollection; 75​ KActionCollection* mActionCollection;
76 DocumentView* mView; 76​ DocumentView* mView;
77 ZoomWidget* mZoomWidget; 77​ ZoomWidget* mZoomWidget;
78 SlideContainer* mToolContainer; 78​ SlideContainer* mToolContainer;
79 ToolContainerContent* mToolContainerContent; 79​ ToolContainerContent* mToolContainerContent;
80 80​
81 QAction * mZoomToFitAction; 81​ QAction * mZoomToFitAction;
82 QAction * mZoomToFitWidthAction; 82​ QAction * mZoomToFillAction;
83 QAction * mActualSizeAction; 83​ QAction * mActualSizeAction;
84 QAction * mZoomInAction; 84​ QAction * mZoomInAction;
85 QAction * mZoomOutAction; 85​ QAction * mZoomOutAction;
86 QList<QAction *> mActions; 86​ QList<QAction *> mActions;
87 87​
88 void setupActions() 88​ void setupActions()
89 { 89​ {
90 KActionCategory* view = new KActionCategory(i18nc("@title actions category - means actions changing smth in interface", "View"), mActionCollection); 90​ KActionCategory* view = new KActionCategory(i18nc("@title actions category - means actions changing smth in interface", "View"), mActionCollection);
91 91​
92 mZoomToFitAction = view->addAction("view_zoom_to_fit"); 92​ mZoomToFitAction = view->addAction("view_zoom_to_fit");
93 view->collection()->setDefaultShortcut(mZoomToFitAction, Qt::Key_F); 93​ view->collection()->setDefaultShortcut(mZoomToFitAction, Qt::Key_F);
94 mZoomToFitAction->setCheckable(true); 94​ mZoomToFitAction->setCheckable(true);
95 mZoomToFitAction->setChecked(true); 95​ mZoomToFitAction->setChecked(true);
96 mZoomToFitAction->setText(i18n("Zoom to Fit")); 96​ mZoomToFitAction->setText(i18n("Zoom to fit"));
97 mZoomToFitAction->setIcon(QIcon::fromTheme("zoom-fit-best")); 97​ mZoomToFitAction->setIcon(QIcon::fromTheme("zoom-fit-best"));
98 mZoomToFitAction->setIconText(i18nc("@action:button Zoom to fit, shown in status bar, keep it short please", "Fit")); 98​ mZoomToFitAction->setIconText(i18nc("@action:button Zoom to fit, shown in status bar, keep it short please", "Fit"));
99 99​
100 mZoomToFitWidthAction = view->addAction("view_zoom_to_fit_width"); 100​ mZoomToFillAction = view->addAction("view_zoom_to_fill");
101 //view->collection()->setDefaultShortcuts(mZoomToFitWidthAction, Qt::Key_W); ?? 101​ //view->collection()->setDefaultShortcuts(mZoomToFillAction, Qt::Key_W); ??
102 mZoomToFitWidthAction->setCheckable(true); 102​ mZoomToFillAction->setCheckable(true);
103 mZoomToFitWidthAction->setChecked(false); 103​ mZoomToFillAction->setChecked(false);
104 mZoomToFitWidthAction->setText(i18n("Zoom to Fit Width")); 104​ mZoomToFillAction->setText(i18n("Zoom to fill window by fitting to width or height"));
rkflxUnsubmitted
Not Done

Could you try to come up with a better tooltip, more akin to the one for 100%?

rkflx: Could you try to come up with a better tooltip, more akin to the one for {nav 100%}?
slenzAuthorUnsubmitted
Not Done

Would it not be better to change all tooltips to something like Zoom to fit/fill screen and Zoom to actual size (Lightroom actually uses "Zoom to 100%")? I can't come up with something good that is similar to the one for 100%.

Slightly off topic, but I don't think anyone actually thinks of themselves as viewing "documents" in an image viewer, so that standard text suits us quite badly anyways.

slenz: Would it not be better to change all tooltips to something like `Zoom to fit/fill screen` and…
rkflxUnsubmitted
Not Done

documents

Yeah, that's not ideal. What I meant was not the text, but the fact the explanation is longer.

IMO the tooltips serve multiple purposes:

  • Mention that the button will "Zoom", because there is no space for that on the buttom. (That's achieved already.)
  • Clarify what "Fit" and "Fill" actually are, because not every user might understand the concepts and some are scared to break something if they click on it. Experts obviously won't bother with the tooltips at all. (That's where we can improve.)

Let's keep it for now as is, perhaps later we'll have a better idea for the wording. I don't want to only repeat what's on the button, but phrase it differently.

Lightroom

Gwenview's target user base is very different, I feel Darktable or RawTherapee users are more technical than the average Gwenview user.

rkflx: > documents Yeah, that's not ideal. What I meant was not the text, but the fact the…
muhlenpfordtUnsubmitted
Not Done

What about e.g. "Zoom to fit smaller side" as tooltip?

muhlenpfordt: What about e.g. "Zoom to fit smaller side" as tooltip?
rkflxUnsubmitted
Not Done

Good idea.

smaller → narrower?

rkflx: Good idea. smaller → narrower?
slenzAuthorUnsubmitted
Not Done

Not really true? Would depend on the current combination of window aspect ratio to image aspect ratio (4:3 image on a 16:9 window does fit width even if that is bigger).

slenz: Not really true? Would depend on the current combination of window aspect ratio to image aspect…
rkflxUnsubmitted
Not Done

I love your attention to detail ;)

How about "Zoom to fill screen by fitting to width or height"

rkflx: I love your attention to detail ;) How about "Zoom to fill screen by fitting to width or…
rkflxUnsubmitted
Not Done

Also: While 3e10699ac37c did not add anything to doc/index.docbook (grep for slider at the bottom), I think it might be worth explaining what the button does, as it may not be obvious to every user unless they are adventurous enough to Rotate when experimenting.

rkflx: Also: While 3e10699ac37c did not add anything to `doc/index.docbook` (grep for `slider at the…
slenzAuthorUnsubmitted
Not Done

Updated but untested. I can't get khelpcenter to actually use my new documentation instead. Perhaps some shell variables I'm missing?

slenz: Updated but untested. I can't get khelpcenter to actually use my new documentation instead.
rkflxUnsubmitted
Not Done

We should improve our documentation on that, for now see here: https://phabricator.kde.org/D10295#206872

rkflx: We should improve our documentation on that, for now see here: https://phabricator.kde.
rkflxUnsubmitted
Not Done

Can you make that "window" instead of "screen"? (Just like you proposed in the docbook.)

rkflx: Can you make that "window" instead of "screen"? (Just like you proposed in the docbook.)
105 mZoomToFitWidthAction->setIcon(QIcon::fromTheme("zoom-fit-best")); 105​ mZoomToFillAction->setIcon(QIcon::fromTheme("zoom-fit-best"));
106 mZoomToFitWidthAction->setIconText(i18nc("@action:button Zoom to fit width, shown in status bar, keep it short please", "Fit Width")); 106​ mZoomToFillAction->setIconText(i18nc("@action:button Zoom to fill (fit width or height), shown in status bar, keep it short please", "Fill"));
rkflxUnsubmitted
Done

Perhaps add (fit width or height) after to fill?

rkflx: Perhaps add `(fit width or height)` after `to fill`?
107 107​
108 mActualSizeAction = view->addAction(KStandardAction::ActualSize); 108​ mActualSizeAction = view->addAction(KStandardAction::ActualSize);
109 mActualSizeAction->setCheckable(true); 109​ mActualSizeAction->setCheckable(true);
110 mZoomToFitWidthAction->setChecked(false); 110​ mZoomToFillAction->setChecked(false);
111 mActualSizeAction->setIcon(QIcon::fromTheme("zoom-original")); 111​ mActualSizeAction->setIcon(QIcon::fromTheme("zoom-original"));
112 mActualSizeAction->setIconText(i18nc("@action:button Zoom to original size, shown in status bar, keep it short please", "100%")); 112​ mActualSizeAction->setIconText(i18nc("@action:button Zoom to original size, shown in status bar, keep it short please", "100%"));
113 113​
114 mZoomInAction = view->addAction(KStandardAction::ZoomIn); 114​ mZoomInAction = view->addAction(KStandardAction::ZoomIn);
115 mZoomOutAction = view->addAction(KStandardAction::ZoomOut); 115​ mZoomOutAction = view->addAction(KStandardAction::ZoomOut);
116 116​
117 mActions << mZoomToFitAction << mActualSizeAction << mZoomInAction << mZoomOutAction << mZoomToFitWidthAction; 117​ mActions << mZoomToFitAction << mActualSizeAction << mZoomInAction << mZoomOutAction << mZoomToFillAction;
118 } 118​ }
119 119​
120 void connectZoomWidget() 120​ void connectZoomWidget()
121 { 121​ {
122 if (!mZoomWidget || !mView) { 122​ if (!mZoomWidget || !mView) {
123 return; 123​ return;
124 } 124​ }
125 125​
▲ Show 20 Lines • Show All 55 Lines • ▼ Show 20 Line(s)172{
181 181​
182 // Connect new view 182​ // Connect new view
183 d->mView = view; 183​ d->mView = view;
184 if (!d->mView) { 184​ if (!d->mView) {
185 return; 185​ return;
186 } 186​ }
187 connect(d->mView, &DocumentView::adapterChanged, this, &DocumentViewController::slotAdapterChanged); 187​ connect(d->mView, &DocumentView::adapterChanged, this, &DocumentViewController::slotAdapterChanged);
188 connect(d->mView, &DocumentView::zoomToFitChanged, this, &DocumentViewController::updateZoomToFitActionFromView); 188​ connect(d->mView, &DocumentView::zoomToFitChanged, this, &DocumentViewController::updateZoomToFitActionFromView);
189 connect(d->mView, &DocumentView::zoomToFitWidthChanged, this, &DocumentViewController::updateZoomToFitWidthActionFromView); 189​ connect(d->mView, &DocumentView::zoomToFillChanged, this, &DocumentViewController::updateZoomToFillActionFromView);
190 connect(d->mView, &DocumentView::currentToolChanged, this, &DocumentViewController::updateTool); 190​ connect(d->mView, &DocumentView::currentToolChanged, this, &DocumentViewController::updateTool);
191 191​
192 connect(d->mZoomToFitAction, SIGNAL(toggled(bool)), 192​ connect(d->mZoomToFitAction, SIGNAL(toggled(bool)),
193 d->mView, SLOT(setZoomToFit(bool))); 193​ d->mView, SLOT(setZoomToFit(bool)));
194 connect(d->mZoomToFitWidthAction, SIGNAL(toggled(bool)), 194​ connect(d->mZoomToFillAction, SIGNAL(toggled(bool)),
195 d->mView, SLOT(setZoomToFitWidth(bool))); 195​ d->mView, SLOT(setZoomToFill(bool)));
196 connect(d->mActualSizeAction, SIGNAL(triggered()), 196​ connect(d->mActualSizeAction, SIGNAL(triggered()),
197 d->mView, SLOT(zoomActualSize())); 197​ d->mView, SLOT(zoomActualSize()));
198 connect(d->mZoomInAction, SIGNAL(triggered()), 198​ connect(d->mZoomInAction, SIGNAL(triggered()),
199 d->mView, SLOT(zoomIn())); 199​ d->mView, SLOT(zoomIn()));
200 connect(d->mZoomOutAction, SIGNAL(triggered()), 200​ connect(d->mZoomOutAction, SIGNAL(triggered()),
201 d->mView, SLOT(zoomOut())); 201​ d->mView, SLOT(zoomOut()));
202 202​
203 d->updateActions(); 203​ d->updateActions();
204 updateZoomToFitActionFromView(); 204​ updateZoomToFitActionFromView();
205 updateZoomToFitWidthActionFromView(); 205​ updateZoomToFillActionFromView();
206 updateTool(); 206​ updateTool();
207 207​
208 // Sync zoom widget 208​ // Sync zoom widget
209 d->connectZoomWidget(); 209​ d->connectZoomWidget();
210 d->updateZoomWidgetVisibility(); 210​ d->updateZoomWidgetVisibility();
211} 211​}
212 212​
213DocumentView* DocumentViewController::view() const 213​DocumentView* DocumentViewController::view() const
214{ 214​{
215 return d->mView; 215​ return d->mView;
216} 216​}
217 217​
218void DocumentViewController::setZoomWidget(ZoomWidget* widget) 218​void DocumentViewController::setZoomWidget(ZoomWidget* widget)
219{ 219​{
220 d->mZoomWidget = widget; 220​ d->mZoomWidget = widget;
221 221​
222 d->mZoomWidget->setActions( 222​ d->mZoomWidget->setActions(
223 d->mZoomToFitAction, 223​ d->mZoomToFitAction,
224 d->mActualSizeAction, 224​ d->mActualSizeAction,
225 d->mZoomInAction, 225​ d->mZoomInAction,
226 d->mZoomOutAction, 226​ d->mZoomOutAction,
227 d->mZoomToFitWidthAction 227​ d->mZoomToFillAction
228 ); 228​ );
229 229​
230 d->mZoomWidget->setMaximumZoom(qreal(DocumentView::MaximumZoom)); 230​ d->mZoomWidget->setMaximumZoom(qreal(DocumentView::MaximumZoom));
231 231​
232 d->connectZoomWidget(); 232​ d->connectZoomWidget();
233 d->updateZoomWidgetVisibility(); 233​ d->updateZoomWidgetVisibility();
234} 234​}
235 235​
Show All 9 Lines
245} 245​}
246 246​
247void DocumentViewController::updateZoomToFitActionFromView() 247​void DocumentViewController::updateZoomToFitActionFromView()
248{ 248​{
249 SignalBlocker blocker(d->mZoomToFitAction); 249​ SignalBlocker blocker(d->mZoomToFitAction);
250 d->mZoomToFitAction->setChecked(d->mView->zoomToFit()); 250​ d->mZoomToFitAction->setChecked(d->mView->zoomToFit());
251} 251​}
252 252​
253void DocumentViewController::updateZoomToFitWidthActionFromView() 253​void DocumentViewController::updateZoomToFillActionFromView()
254{ 254​{
255 SignalBlocker blocker(d->mZoomToFitWidthAction); 255​ SignalBlocker blocker(d->mZoomToFillAction);
256 d->mZoomToFitWidthAction->setChecked(d->mView->zoomToFitWidth()); 256​ d->mZoomToFillAction->setChecked(d->mView->zoomToFill());
257} 257​}
258 258​
259void DocumentViewController::updateTool() 259​void DocumentViewController::updateTool()
260{ 260​{
261 if (!d->mToolContainer) { 261​ if (!d->mToolContainer) {
262 return; 262​ return;
263 } 263​ }
264 AbstractRasterImageViewTool* tool = d->mView->currentTool(); 264​ AbstractRasterImageViewTool* tool = d->mView->currentTool();
Show All 18 Lines