renatoo added inline comments.

INLINE COMMENTS

> ervin wrote in kfileplacesmodel.cpp:476
> Either reorder the enum or change for a different (more explicit not relying 
> on enum storage) comparison operator to have the "right" order. By right 
> order I assume the goal was to align with Dolphin which does: Places / 
> Recently Saved / Search For / Devices.
> 
> Currently we have a different order:  Places / Search For / Recently Saved / 
> Devices.

Are you fine If I set values for enums? Something like:

  enum GroupType
  {
      PlacesType = 100,
      RecentlySavedType = 200,
      SearchForType = 300,
      DevicesType = 400,
      RemovableDeviceType = 500,
      NetworkType = 600
  };

REPOSITORY
  R241 KIO

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

To: renatoo, #frameworks, dfaure, ervin, #vdg
Cc: markg, ngraham, mlaurent, ervin, anthonyfieroni, cfeck, #frameworks

Reply via email to