> On June 17, 2016, 5:12 p.m., Tomasz Janiszewski wrote: > > site/source/layouts/basic.erb, line 7 > > <https://reviews.apache.org/r/48861/diff/1/?file=1422500#file1422500line7> > > > > How about adding other tags `og:site_name` and `og:title`. By default > > type is set to website is that correct for docs and blog posts? > > haosdent huang wrote: > Haha, actually facebook standard require `og:type`, `og:url`, `og:title`, > `og:description`, `fb:app_id` as well. > However, I found the facebook preview works if just set `og:image` and I > try to avoid bring too much unnecessary `meta` tags which only work for > facebook. > Do you think this make sense? I could continue to add `og:site_name` and > `og:title` if you think they are necessary. Just need add two lines. :-) > > haosdent huang wrote: > Looks like twitter use `<meta name="twitter:xxx` and facebook use > opengraph `<meta name="op:` created by them. > > Tomasz Janiszewski wrote: > Twitter also supports opengraph but require twitter:card and twitter:site > tags to work. I think opengraph is now a defacto standard supported by all > social media sites. > > Tomasz Janiszewski wrote: > On the other hand I see if no title is provided then title tag is used. > I'm not sure what is taken as a description but assume it'll be the first > paragraph so we could leave it as is.
Cool, thank you for detail explanation! - haosdent ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48861/#review138246 ----------------------------------------------------------- On June 17, 2016, 4:55 p.m., haosdent huang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48861/ > ----------------------------------------------------------- > > (Updated June 17, 2016, 4:55 p.m.) > > > Review request for mesos, Alexander Rukletsov, Tomasz Janiszewski, Jie Yu, > and Vinod Kone. > > > Bugs: MESOS-5617 > https://issues.apache.org/jira/browse/MESOS-5617 > > > Repository: mesos > > > Description > ------- > > Added preview logo for facebook. > > > Diffs > ----- > > site/source/assets/img/mesos_logo_fb_preview.png PRE-CREATION > site/source/layouts/basic.erb d654cab802dd60bc70bafac124d411048800fb7a > > Diff: https://reviews.apache.org/r/48861/diff/ > > > Testing > ------- > > Example: >  > > > Thanks, > > haosdent huang > >
