person picker in bug reports hides the header for non affiliated people

Bug #803990 reported by Diogo Matsubara
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Ian Booth

Bug Description

Steps to reproduce:
1. Log in with an user without any privileges or team memberships (e.g. ~dm-lptest)
2. Open https://bugs.qastaging.launchpad.net/launchpad/+bug/2112
3. Click the assigned to edit button. Notice there's only the assign to me link. See no-header-in-picker.png
4. Close it
5. Click the assigned to edit button again.

What happens:
The header "Select a team which you are a member" as well as the green line appears. See picker-with-header.png

I was unable to reproduce this bug on launchpad.dev.

Related branches

Revision history for this message
Diogo Matsubara (matsubara) wrote :
Revision history for this message
Diogo Matsubara (matsubara) wrote :
Revision history for this message
William Grant (wgrant) wrote :

This is deliberate:

        if (conf.hide_assignee_team_selection) {
            content_box = assignee_picker.get('contentBox');
            search_box = content_box.one('.yui3-picker-search-box');
            search_box.setStyle('display', 'none');
            step_title = content_box.one('.contains-steptitle');
            step_title.setStyle('display', 'none');
        }

I agree it looks a bit strange, but I'm not sure what it should say.

Revision history for this message
Curtis Hovey (sinzui) wrote :

I think the green line between the header and the instructions must be shown. The instructions could say:
    You may only assign yourself because you are not affiliated with this project and do not have any team memberships.

Ian Booth (wallyworld)
Changed in launchpad:
assignee: nobody → Ian Booth (wallyworld)
status: Triaged → In Progress
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :
tags: added: qa-needstesting
Changed in launchpad:
status: In Progress → Fix Committed
William Grant (wgrant)
tags: added: qa-ok
removed: qa-needstesting
William Grant (wgrant)
Changed in launchpad:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.