<feed xmlns='http://www.w3.org/2005/Atom'>
<title>dolphin/src/dolphintabpage.cpp, branch master</title>
<subtitle>Patched KDE Dolphin with Pixel Scaling
</subtitle>
<id>https://fiftyfourth.xyz/git/dolphin/atom?h=master</id>
<link rel='self' href='https://fiftyfourth.xyz/git/dolphin/atom?h=master'/>
<link rel='alternate' type='text/html' href='https://fiftyfourth.xyz/git/dolphin/'/>
<updated>2026-01-10T15:14:26Z</updated>
<entry>
<title>Use Q_ASSERT instead of Q_CHECK_PTR for plain checks of pointer value</title>
<updated>2026-01-10T15:14:26Z</updated>
<author>
<name>Friedrich W. H. Kossebau</name>
<email>kossebau@kde.org</email>
</author>
<published>2026-01-09T15:16:48Z</published>
<link rel='alternate' type='text/html' href='https://fiftyfourth.xyz/git/dolphin/commit/?id=5fff1e414a060c0619a97ad8d3f5665e3f5a4878'/>
<id>urn:sha1:5fff1e414a060c0619a97ad8d3f5665e3f5a4878</id>
<content type='text'>
Q_CHECK_PTR (other than the name suggests) is intended only to check the
success of memory allocations.
Using it for any pointers instead can be misleading due to the
"Out of memory" log printed in case of hits.
</content>
</entry>
<entry>
<title>dolphintabbpage: Allow migration for users with the old session file format</title>
<updated>2025-12-23T10:02:05Z</updated>
<author>
<name>Oliver Schramm</name>
<email>oliver@oliver-schramm.eu</email>
</author>
<published>2025-12-23T10:02:05Z</published>
<link rel='alternate' type='text/html' href='https://fiftyfourth.xyz/git/dolphin/commit/?id=bd20879e3bbb42fc3a11ec90ba67db32b9b62f6d'/>
<id>urn:sha1:bd20879e3bbb42fc3a11ec90ba67db32b9b62f6d</id>
<content type='text'>
Commit 8d44699f introduced a new session file format, without providing a migration path for the old file format, which led to users losing their session.

Besides adding the migration path, the condition check for future updates is necessary, since we don't provide guarantees for backwards-compatible format changes. In case users downgrade their Dolphin version/immutable distro or they share the Dolphin config file across multiple systems, Dolphin at least won't crash when reading a session file with a newer format. 

BUG: 513466
</content>
</entry>
<entry>
<title>folderstabssettingspage: add 3 options for closing split view</title>
<updated>2025-12-03T11:19:46Z</updated>
<author>
<name>Rafał Lichwała</name>
<email>rafal@siliconet.pl</email>
</author>
<published>2025-11-29T12:32:30Z</published>
<link rel='alternate' type='text/html' href='https://fiftyfourth.xyz/git/dolphin/commit/?id=9e5926147606737a87cf17535db1ecbfd2c9a2c5'/>
<id>urn:sha1:9e5926147606737a87cf17535db1ecbfd2c9a2c5</id>
<content type='text'>
There is one bool setting in Dolphin Interface-&gt;Folders&amp;Tabs-&gt;Split_view
which determines which panel is closed on "Close split view" button
click (currently focused or unfocused one).

This commit replaces this setting with (enum-based) 3-state combo box.
Since now we can set "When closing a split view" action to:
 - Close the active pane
 - Close the inactive pane
 - Close the right pane

BUG: 464696
BUG: 186185
</content>
</entry>
<entry>
<title>dolphinnavigatorswidgetaction: show focus indicator in split mode</title>
<updated>2025-11-27T10:29:38Z</updated>
<author>
<name>Rafał Lichwała</name>
<email>rafal@siliconet.pl</email>
</author>
<published>2025-11-27T10:29:38Z</published>
<link rel='alternate' type='text/html' href='https://fiftyfourth.xyz/git/dolphin/commit/?id=2adbecd269e3694c2796374260c1983a8d21764d'/>
<id>urn:sha1:2adbecd269e3694c2796374260c1983a8d21764d</id>
<content type='text'>
This commit uses KUrlNavigator focus indicator to show sufficiently which panel (left or right) is currently active in split mode.

This MR is based on KIO changes in https://invent.kde.org/frameworks/kio/-/merge_requests/2054

See discussion in MR: !1076 (comment 1334290), see https://invent.kde.org/frameworks/kio/-/merge_requests/2058

@teams/vdg

![Screenshot_20251117_232634.png](/uploads/2999f35fdc1c6dd359621c60d23460f2/Screenshot_20251117_232634.png){width=849 height=313}

![Screenshot_20251117_232704.png](/uploads/bd7b75f9159ab533b8e446781233e027/Screenshot_20251117_232704.png){width=849 height=313}

![Screenshot_20251117_232740.png](/uploads/e07153c1e9ae162ebc0ff3f09df1ac34/Screenshot_20251117_232740.png){width=849 height=313}

![Screenshot_20251117_232752.png](/uploads/ececd4c70c0e1f67145e2486d1cf21f2/Screenshot_20251117_232752.png){width=849 height=313}

@teams/vdg

BUG: 150718

BUG: 428440
</content>
</entry>
<entry>
<title>dolphintabpage: save and restore splitter position</title>
<updated>2025-10-31T10:19:48Z</updated>
<author>
<name>Rafał Lichwała</name>
<email>rafal@siliconet.pl</email>
</author>
<published>2025-10-30T15:11:43Z</published>
<link rel='alternate' type='text/html' href='https://fiftyfourth.xyz/git/dolphin/commit/?id=8d44699fa48bc4028b2a7fab4af77ad91f899956'/>
<id>urn:sha1:8d44699fa48bc4028b2a7fab4af77ad91f899956</id>
<content type='text'>
This commit changes version number for save/restore tab state,
because format has changed - m_splitterLastPosition is now also saved/restored

BUG: 430705
</content>
</entry>
<entry>
<title>dolphintabpage: when enabling split view without animation reset sizes</title>
<updated>2025-10-28T09:45:26Z</updated>
<author>
<name>Méven Car</name>
<email>meven@kde.org</email>
</author>
<published>2025-10-28T09:45:26Z</published>
<link rel='alternate' type='text/html' href='https://fiftyfourth.xyz/git/dolphin/commit/?id=71bdbb87dde6ea6255b687af255ead75691d95c5'/>
<id>urn:sha1:71bdbb87dde6ea6255b687af255ead75691d95c5</id>
<content type='text'>
of the splitter.

So the previous size of a pontential leftover closed state does not
apply initially.

BUG: 510915
</content>
</entry>
<entry>
<title>DolphinTabPage, DolphinView: Fix duplicate folder for MTP device</title>
<updated>2025-10-03T06:49:14Z</updated>
<author>
<name>Wendi Gan</name>
<email>gwdx@mail.ustc.edu.cn</email>
</author>
<published>2025-09-02T02:38:32Z</published>
<link rel='alternate' type='text/html' href='https://fiftyfourth.xyz/git/dolphin/commit/?id=d0f8985b4c5c790781be6fcd06d299f087e78756'/>
<id>urn:sha1:d0f8985b4c5c790781be6fcd06d299f087e78756</id>
<content type='text'>
Issue:
When opening an MTP device via the sidebar, Dolphin shows one "Internal shared storage" folder. However, clicking the entry again results in a duplicate folder being displayed.
This regression was introduced by !865 (commit 6c7c047).

Reason:
The URL of an MTP device in the sidebar is `mtp:udi=/org/kde/solid/udev/sys/devices/...`. When clicking to open the MTP device, it is redirected to `mtp:/...`.
- On the first click (no cache), `KCoreDirListerCache::slotUpdateResult()` calls `KCoreDirListerPrivate::emitItems()`.
- On the second click (with cache), `KCoreDirListerCache::slotRedirection()` first calls `KCoreDirListerPrivate::emitItems()`. Besides, `DolphinTabPage::slotViewUrlRedirection()` calls `KCoreDirLister::openUrl()`, which triggers `CachedItemsJob::start()`, and eventually calls `KCoreDirListerPrivate::emitItems()` again. As a result, two `KCoreDirLister::itemsAdded` signals are emitted, causing `m_pendingItemsToInsert` to be appended twice.

---

BUG 496414:
Need to rename folder twice to make it reflect in the tab title.

Reason:
`m_url` is updated after emitting the `DolphinView::redirection` signal. It triggers `DolphinTabWidget::tabUrlChanged()`, which still uses the old URL and thus resets the tab title incorrectly.

---

Change:
- Revert the change in `DolphinTabPage::slotViewUrlRedirection()`.
- Update `m_url` before `DolphinView::redirection` to fix BUG 496414.
- Emit `DolphinView::urlChanged` signal to refresh the navigator of the inactive view.

BUG: 506634
CCBUG: 497313
CCBUG: 496414
</content>
</entry>
<entry>
<title>DolphinTabPage: Show separator when navigator is outside of menubar</title>
<updated>2025-09-19T08:22:23Z</updated>
<author>
<name>Akseli Lahtinen</name>
<email>akselmo@akselmo.dev</email>
</author>
<published>2025-09-19T08:22:23Z</published>
<link rel='alternate' type='text/html' href='https://fiftyfourth.xyz/git/dolphin/commit/?id=2a0fa83b72631b21d2e8d731feb0eff576af0f77'/>
<id>urn:sha1:2a0fa83b72631b21d2e8d731feb0eff576af0f77</id>
<content type='text'>
When navigator is outside of menubar, we should show separator under
it to make it visually easier to understand it's a clickable area.

This adds a small separator, that will be disabled when
the navigator is moved back to menubar.

BUG: 508303
</content>
</entry>
<entry>
<title>changed all calls to "title(), setTitle()" to "customLabel(), setCustomLabel()"</title>
<updated>2025-02-06T14:32:35Z</updated>
<author>
<name>ambar chakravartty</name>
<email>amch9605@gmail.com</email>
</author>
<published>2025-02-02T05:06:06Z</published>
<link rel='alternate' type='text/html' href='https://fiftyfourth.xyz/git/dolphin/commit/?id=f1a5aaa183e4e917a6d1b19fa328fca0a38df479'/>
<id>urn:sha1:f1a5aaa183e4e917a6d1b19fa328fca0a38df479</id>
<content type='text'>
</content>
</entry>
<entry>
<title>changes saveState() and restoreState() to restore renamed tabs (if any)</title>
<updated>2025-02-06T14:32:35Z</updated>
<author>
<name>ambar chakravartty</name>
<email>chakravarttyambar@gmail.com</email>
</author>
<published>2025-01-30T11:59:32Z</published>
<link rel='alternate' type='text/html' href='https://fiftyfourth.xyz/git/dolphin/commit/?id=0bf31bd847d28485d3a2115b0ce9e794654a053b'/>
<id>urn:sha1:0bf31bd847d28485d3a2115b0ce9e794654a053b</id>
<content type='text'>
</content>
</entry>
</feed>
