Modify

Opened 7 years ago

Last modified 7 years ago

#13278 new defect

Not working in different permission roles

Reported by: ntmlod Owned by: falkb
Priority: normal Component: ComponentHierarchyPlugin
Severity: normal Keywords:
Cc: Trac Release: 1.0

Description

After installation and components configuration, I have trouble to get this going in every cases:

  • as a common user ('authenticated' grp + TICKET_CHGPROP), default list
  • as a common developer (grp with TICKET_ADMIN), expected hierarchy list
  • as administrator (TRAC_ADMIN), back to default list

So doesn't seem to be related to permissions and nothing in the log.

I suspect something related to the environment of the page (Js, CSS, ...) because the drop-down menu looks like a bit different (distinct background color and font style).

Attachments (1)

t13278_select.diff (83.7 KB) - added by Ryan J Ollos 7 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 7 years ago by Ryan J Ollos

Any errors in the browser console? (TracTroubleshooting#ChecktheBrowserConsole)

comment:2 Changed 7 years ago by ntmlod

Yeah I had checked it a little already, nothing really serious for me.

Under 'Admin' some messages from gridmodify.js (GridModifyPlugin). No warning under "User"

comment:3 Changed 7 years ago by Ryan J Ollos

I've not found any styling that differs by user permissions, but I have found styling that differs by browser:

  • Works in FF 55
  • Doesn't work in FF57b3
  • Doesn't work in Chrome 61

From a quick read on StackOverflow, it appears it may not be possible to style the HTML option elements in all browsers.

Changed 7 years ago by Ryan J Ollos

Attachment: t13278_select.diff added

comment:4 Changed 7 years ago by Ryan J Ollos

We could use the select2 library to style the elements: t13278_select.diff.

comment:5 Changed 7 years ago by ntmlod

Well on my side I use the same FF 52 but on 3 different sessions ('User', 'Developer' and 'Admin').

I make use of 'User' & 'Admin' only for testing and project management activities. These are very clean sessions with no particular addons installed that could alter the browser behavior. Whereas 'Developer' is my routine session with some securities modules.

I switched to Chrome (only in 38 version), no different session but the styling doesn't work on any roles.

comment:6 Changed 7 years ago by ntmlod

Thanks, your patch works.

It's funny because it don't even know the svn patch subcommand and our forge server, which is not as outdated as I said it was, has a newer SVN installation than my local one. In fact, few modifications have not been imported so I finished it manually.

Maybe still some styling optimizations to do for Chrome but nothing that I can't do on my own.

comment:7 Changed 7 years ago by Ryan J Ollos

I noticed some issues with select box width. Feel free to contribute back another patch that improves styling.

I'm surprised the patch would not apply cleanly as it's based on the latest 0.12 branch.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain falkb.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.