D25987: Don't overload the word "Properties" to mean multiple things

Méven Car noreply at phabricator.kde.org
Tue Dec 17 07:41:02 GMT 2019


meven added inline comments.

INLINE COMMENTS

> viewpropertiesdialog.cpp:51
>  
>  ViewPropertiesDialog::ViewPropertiesDialog(DolphinView* dolphinView) :
>      QDialog(dolphinView),

It'd be great to rename the related class and files accordingly to avoid having a naming gap between devs and users.

REPOSITORY
  R318 Dolphin

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

To: ngraham, #vdg, #dolphin, ndavis, elvisangelaccio
Cc: meven, ndavis, kfm-devel, kde-doc-english, pberestov, iasensio, fprice, gennad, MrPepe, fbampaloukas, alexde, Codezela, feverfew, spoorun, navarromorales, firef, ngraham, andrebarros, skadinna, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20191217/82bfa1fb/attachment.htm>


More information about the kfm-devel mailing list