User needs to access the ToC Settings Panel

RESOLVED FIXED

Status

Hello (Loop)
Client
P1
normal
Rank:
27
RESOLVED FIXED
2 years ago
a year ago

People

(Reporter: fcampo, Assigned: fcampo)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [akita-beta])

Attachments

(2 attachments, 2 obsolete attachments)

(Assignee)

Description

2 years ago
When clicking the 'Settings' button (represented by the gear image), the Settings Panel should be displayed, giving access to the options:

- Change Room Name  || ONLY ROOM OWNER
    Focus on the Room Name field, that becomes editable (same as with click)

- Change my Username
    Takes the user to the specific FTU step where a new Username can be set

- Turn Notifications Off/On
    As it says, toggles the notifications ON or OFF

- Take the Tour
    Relaunches the whole FTU

- Submit Feedback
    Opens a New Tab with the Feedback screen

- Help
    Opens a New Tab with the Firefox Hello Help page
(Assignee)

Updated

2 years ago
Blocks: 1268042, 1268124
(Assignee)

Updated

2 years ago
Points: --- → 2
Whiteboard: [triage]
Blocks: 1273597
Rank: 27
Priority: -- → P2
Whiteboard: [triage] → [akita]
Whiteboard: [akita] → [akita-beta]
(Assignee)

Updated

2 years ago
Assignee: nobody → fernando.campo

Comment 1

2 years ago
Created attachment 8761378 [details] [review]
[loop] fcampo:settings-panel-shared-options > mozilla:akita
(Assignee)

Comment 2

2 years ago
Comment on attachment 8761378 [details] [review]
[loop] fcampo:settings-panel-shared-options > mozilla:akita

- Moving settings panel to shared views
- Modify options to follow wireframes (but keeping FxAccounts until we decide what to do with it)
- Adding functionality for the standalone side
Attachment #8761378 - Flags: review?(standard8)
Attachment #8761378 - Flags: review?(dpreston)
Attachment #8761378 - Flags: review?(dmose)
Attachment #8761378 - Flags: review?(crafuse)
Attachment #8761378 - Flags: review?(standard8)
Attachment #8761378 - Flags: review?(dpreston)
Attachment #8761378 - Flags: review?(crafuse)

Comment 3

a year ago
Created attachment 8766983 [details] [review]
[loop] dmose:fcampo-settings-panel-shared-options > mozilla:akita
Comment on attachment 8761378 [details] [review]
[loop] fcampo:settings-panel-shared-options > mozilla:akita

Obsoleted by rebased version of PR.
Attachment #8761378 - Attachment is obsolete: true
Attachment #8761378 - Flags: review?(dmose)
I've rebased/unconflicted and posted a new PR.  Marking this as blocking akita-catfood, as it creates the UI that allows guests to set their own nicknames.  There seems to be a problem where it doesn't correctly hide some of the items on the guest side, however.
Blocks: 1283558
Priority: P2 → P1
(Assignee)

Comment 6

a year ago
Hey Dan, I can take a look at it if you want, which one is not being hidden correctly?

Should we add reviewers to the PR?
Flags: needinfo?(dmose)
I can't easily check it from here, but I believe none of them were being hidden.

Adding reviewers sounds like a fine idea.
Flags: needinfo?(dmose)
(Assignee)

Comment 8

a year ago
Created attachment 8768832 [details]
ToC-settings_guest_vs_owner

Dan, can you give more details about the options not hiding on the guest side?

I tested the updated patch, and the only entry that should be hiding for guests (change room name) does not show up.

For the rest, even thought functionality is not implemented, no rules for showing/hiding are set, as far as I know
Flags: needinfo?(dmose)
You're right; I'm seeing the same thing.  That said, on my machine, at least, it appears that none of the entries in the menu actually do anything, and Tour, Feedback, and Sign In also don't close the menu when you click on them.  So I guess I was sort of expecting those to be hidden.

At this point, I think it's really your call on what to land...
Flags: needinfo?(dmose)
(Assignee)

Comment 10

a year ago
Well, it makes sense to, at least, disable the options that have no functionality, if not hidden.

Will do that, and merge.Thanks.

Comment 11

a year ago
Created attachment 8769116 [details] [review]
[loop] fcampo:settings-panel-shared-options > mozilla:akita
(Assignee)

Comment 12

a year ago
Comment on attachment 8766983 [details] [review]
[loop] dmose:fcampo-settings-panel-shared-options > mozilla:akita

Adding new PR with:
- disabled state for entries with no functionality (guest).
- rebase problems fixed.
Attachment #8766983 - Attachment is obsolete: true
(Assignee)

Comment 13

a year ago
merged - https://github.com/mozilla/loop/commit/edd1dc812d943490e785385d067009031ca6eccc

(functional still crashing)
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.