704714 - AppSync UI needs polishing within the Dashboard

Status: RESOLVED INVALID

(Web Apps Graveyard :: AppsInTheCloud, defect, P3)

Description

[Mohamed Dabbagh]

The AppSync UI within the Dashboard needs polishing. Currently, after the user signs in to BrowserID, some click-able text appears but that's about it. The formatting and placement is not correct either as the email gets cut-off if it's long enough.

This bug could be used to track UI discussions for AppSync if needed.

Steps to Reproduce:
1. Launch Firefox
2. Make sure to not have the apps extension installed or enabled - As of the date of this bug being filed, AppSync works only without the extension
3. Go to myapps.mozillalabs.com
4. Sign in to BrowserID - You'll need to create a BrowserID account at https://browserid.org/ if you don't already have one

Actual Result: 
Text appears stating 'Sync'. The UI is not very polished. 

Expected Results:
A polished UI with correct formatting is shown within the Dashboard.

Comment 2

[Ian Bicking (:ianbicking)]

https://github.com/mozilla/openwebapps/commit/00940d057b2d9c65f3d8c119b976651e9ab86f5d

Comment 3

[Ian Bicking (:ianbicking)]

deployed to myapps

Comment 4

[Mohamed Dabbagh]

Attachment: Screenshot

Comment 5

[Mohamed Dabbagh]

There is still a bug where the BrowserID names are cut-off. Please see attachments for a screenshot.

Comment 6

[Ian Bicking (:ianbicking)]

I'm not sure what to do about name cutoffs.  It's kind of constrained to an absolute width, and there's no width where we can be sure the usernames will fit in.  I guess <span title="username">username</a> would make it possible, though I don't know any graceful way to do anything but truncate the username (e.g., it would be nice to have "somelonguser@dom..." but I don't know how to get those ellipsis in the right place).

Someone with better CSS-fu than I might have an idea.

Comment 7

[Mohamed Dabbagh]

Maybe we can move the BrowserID sign-in button and the div related to Sync to the left a little closer to the Dashboard? Right now, it's far to the right and looks out of place. This way we don't have to do anything with the name and it will fully show up since there is more room now. Just a thought.

Comment 8

[Mohamed Dabbagh]

Attachment: Screenshot of suggestion to move sync closer to Dashboard

Comment 9

[Jason Smith [:jsmith]]

A Pivotal Tracker story has been created for this Bug: https://www.pivotaltracker.com/story/show/24808907

Comment 10

[Jason Smith [:jsmith]]

The old app sync codebase is no longer going to be supported. Apps in the cloud will be under a different codebase. None of these bugs are valid anymore.