hein requested changes to this revision. hein added a comment. This revision now requires changes to proceed.
(a) I don't like accumulating data in a cache that's never evicted and never changed for correctness. What if the hierarchy changes in the meantime? This is like a subtle bug trap. (b) It's "Up", not "Back". If there's no way up from a location then there's no way up. Maybe for links we should make it "Back", and just keep the previous URL around. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D3931 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: chinmoyr, #plasma, hein Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
