Github user mike-jumper commented on a diff in the pull request:

    https://github.com/apache/guacamole-client/pull/301#discussion_r196296995
  
    --- Diff: 
guacamole/src/main/webapp/app/navigation/services/userPageService.js ---
    @@ -312,6 +305,12 @@ 
angular.module('navigation').factory('userPageService', ['$injector',
                 url  : '/settings/preferences'
             }));
     
    +        // Add link to Session management (always accessible)
    +        pages.push(new PageDefinition({
    --- End diff --
    
    Order matters within the `pages` array. Moving the session management tab 
definition to the bottom effectively reorders the tabs of the settings screen 
such that the session management tab is last. Unless there is a reason to 
change the tab order, we should keep it consistent.


---

Reply via email to