fvogt added inline comments.

INLINE COMMENTS

> broulik wrote in content-script.js:333
> An empty title is kinda pointless anyway, so this is on purpose

It would still send an empty title on initialization or changes after the first 
change though. So if that's the purpose of the empty string assignment, it 
should be done differently.

REPOSITORY
  R856 Plasma Browser Integration

REVISION DETAIL
  https://phabricator.kde.org/D23773

To: broulik, #plasma, fvogt, ognarb
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, 
ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart

Reply via email to