Move c/b/r/chromeos/quick_unlock/pin_keyboard into settings. |
||||
Issue descriptionSince kShowNonMdLogin has been recently removed, settings is the only user of pin_keyboard.html/js. It shares quit a bit of code with md_pin_keyboard.html/js but i think those are going away soon-ish, so i don't think there is a need to move common stuff into cr-components or cr-elements.
,
Feb 14 2018
One thing to keep in mind is that soonish we will want to let the user setup PIN in oobe - I'm not sure how existing settings elements are exposed to oobe. stevenjb@ should be familiar though.
,
Feb 14 2018
If the code will be used in OOBE webui then we should go ahead and move it to cr_components. That is where other components shared between OOBE and Settings live. Do we still need: chrome/browser/resources/chromeos/quick_unlock/pin_keyboard.html/js ? There is no reference to it in browser_resources.grd. The login code references chrome://oobe/custom_elements/pin_keyboard.html, but that is mapped to IDR_MD_CUSTOM_ELEMENTS_PIN_KEYBOARD_HTML in oobe_ui.cc. I would prefer not to duplicate code, even "briefly", but especially if we expect to re-introduce the shared component to OOBE.
,
Feb 14 2018
Yes, pin_keyboard.html/js is the one that incorrectly referenced from settings/people_page/pin_keyboard. I'll see what the differences between pin and md_pin and see if I can create a cr_components and add slots and or vars for the differences.
,
Feb 14 2018
I would expect that we would want the UIs to be consistent, so if there are differences we should just go ahead and migrate the Settings UI to also use the MD version.
,
Feb 14 2018
+tbuckley The MD style does not work right away (white text meant for dark background and large buttons taking up too much space on the skinny dialog for instance). The UI should be redesigned a bit to fit both.
,
Mar 2 2018
|
||||
►
Sign in to add a comment |
||||
Comment 1 by sammiequon@chromium.org
, Feb 14 2018