- Group Reviewers
- R40:f740df010c50: QuickOpenSession: Indicate clearly the active session
I would prefer to not hide it since it's confusing. Either gray out the session by calling action ->setEnabled(false), or mark it as active by e.g. adding a '*' or similar to the name.
In any case, using the range based for loop is certainly bette.