<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sat, Aug 13, 2016 at 6:10 AM, Gijs Kruitbosch <span dir="ltr"><<a href="mailto:gijskruitbosch@gmail.com" target="_blank">gijskruitbosch@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
How would people feel about creating a new (Firefox) bugzilla component for control center? At the moment it seems like all the bugs end up either in General or in Location Bar (and some maybe in Device Permissions?), none of which is a great fit. It seems like it'd make sense for the feature to have its own component </blockquote><div><br>In general I'm in favour of more focused components rather than dumping grounds like Firefox::General and Firefox::Security<div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">​ as I think it makes finding existing bugs and therefore triage easier​. It also makes it easier to prioritize buckets/themes of bugs.<br> </div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">(possibly morphing the Device Permissions one if those are all (going to be) part of CC).<br></blockquote><div><br><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">I'm also in favour of morphing/clarifying the Device Permissions component (which has always been confusing to me) since it has bugs which involve the location bar, control center, and doorhangers but only for device permissions and doesn't include similar permissions which share (some of) the same UI and UI logic (e.g. geolocation and push/web notifications). On the other hand there are enough bugs in the current Device Permissions component that maybe it warrants its own component. Also, a lot of the bugs in this component are about the initial prompt which isn't really control center.<br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default"><br>The component description is "Bugs and feature requests for the Firefox UI
 to give or revoke web page permissions to access devices (camera, 
microphone, screen share, ...)."​</div> <div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">​btw.​</div><br><br><div style="font-family:arial,helvetica,sans-serif" class="gmail_default"><div class="gmail_quote">On Mon, Aug 15, 2016 at 10:11 AM, Brian Grinstead <span dir="ltr"><<a href="mailto:bgrinstead@mozilla.com" target="_blank">bgrinstead@mozilla.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class=""></span><span class=""><br>
</span>It makes sense to me.  I take it the proposed split here is that 
bugs to do with #identity-box belong in Location Bar and bugs to do with
 UI inside the control center popup belong in this new component?<br></blockquote><div><br></div><div>That's one way that would make sense though maybe the permission request anchors shouldn't be part of that since they're actually a different panel? It seems like there are three related UI pieces: <br>* identity block (which is where requests are anchored)<br>* identity panel <br>* permission requests (which use the toolkit PopupNotifications.jsm module with bugs in the Toolkit::Notifications and Alerts component and have some bugs in Firefox::Device Permissions)<br><br>Do we want all three of those UI pieces in one component? I think I lean towards yes since icons in the identity box can sometimes open the identity panel and sometimes be the anchor for a permission request and bug filers probably won't understand the distinction. When the permission icon isn't the anchor of a request (e.g. a strike-through on the geolocation icon) it's synchronized with state in the identity panel and so they probably belong in the same component. With all three of those pieces in one new component I think the only remaining use of the Device Permissions component would be for UI outside the browser window indicating devices in use and so that component could be renamed or removed.<br><br></div><div>Matthew N. (:MattN)<br></div></div></div></div></div></div></div>