Mark Banner (:standard8) Reporter
	Description • 9 years ago

Currently we're storing the thumbnails associated with context as a simple url.

The architecture calls for this to be stored as a data uri - this seems reasonable as it'd prevent issues with the website changing/removing images once someone had added them to a room.

https://wiki.mozilla.org/Loop/Architecture/Context#Format_of_context.value

RT has also said that we probably want to switch to favicons only, so doing that here as well might make it simpler.

	Romain Testard [:RT]
	Comment 1 • 9 years ago

Please let's move to Favicons only.
I copy Sevaan so he's aware - we discussed and it's his preference too.

	Mark Banner (:standard8) Reporter
	Comment 3 • 9 years ago

I've just taken a look at this. Since we're switching to favicons only, we could go simpler on the document title, and just get the title of the current tab.

Mike/Jared, do you think we should:

a) update PageMetaData to also handle favicons, or
b) just get the favicon from the favicon service and use the current document title

?

	Sevaan Franks [:sevaan]
	Comment 4 • 9 years ago

Note, it says "Let's talk about", not "Let's talk about this page".

	Sevaan Franks [:sevaan]
	Comment 5 • 9 years ago

Height of box with contextual information grows if page title wraps.

	Sevaan Franks [:sevaan]
	Comment 6 • 9 years ago

Note: Contextual information is left-aligned, not centered.

	Mark Banner (:standard8) Reporter
	Comment 7 • 9 years ago

Fixed by bug 1162903.