[Differential] [Commented On] D3931: [Plasma folderview] Replace "Up" button with "Back" button in listview mode

[Eike Hein]

hein added a comment.


  Looks like it will be the code comment route:
  
  ‎[17:38] ‎<‎Sho_‎>‎ btw
  ‎[17:39] ‎<‎Sho_‎>‎ i have a dev on phab who claims push/pop on a [] held in a var-type prop doesn't notify
  ‎[17:39] ‎<‎Sho_‎>‎ anyone run into this?
  ‎[17:39] ‎<‎kbroulik‎>‎ yup
  ‎[17:39] ‎<‎kbroulik‎>‎ that's by design™
  ‎[17:39] ‎<‎kbroulik‎>‎ QML can't detect that
  ‎[17:39] ‎<‎Sho_‎>‎ even for a JS array?
  ‎[17:39] ‎<‎kbroulik‎>‎ yup
  ‎[17:39] ‎<‎Sho_‎>‎ or is this just with QML lists
  ‎[17:39] ‎<‎kbroulik‎>‎ also for JS objects
  ‎[17:39] ‎<‎Sho_‎>‎ ok ...
  ‎[17:39] ‎<‎Sho_‎>‎ what did you do to work around?
  ‎[17:39] ‎<‎kbroulik‎>‎ var foo = prop
  ‎[17:39] ‎<‎kbroulik‎>‎ foo.push(bar)
  ‎[17:39] ‎<‎kbroulik‎>‎ prop = foo
  ‎[17:40] ‎<‎Sho_‎>‎ how aful

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D3931

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: chinmoyr, hein, #plasma
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, andreaska, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170109/4a79ec3b/attachment.html>