ognarb added a comment.

  Tested in Firefox:
  
  The PNG is not displayed.
  
  I'm not a fan of using string as enum value (portStatus).

INLINE COMMENTS

> action_popup.css:22
> +
> +section {
> +    padding: 1em;

Not used

> ognarb wrote in action_popup.css:32
> Why it is empty?

Or maybe add a breezy background-color :)

REPOSITORY
  R856 Plasma Browser Integration

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

To: broulik, #plasma, #vdg, fvogt, ognarb
Cc: ognarb, 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