bruns added a comment.

  In D16421#349072 <https://phabricator.kde.org/D16421#349072>, @ngraham wrote:
  
  > After manual application, the alignment is perfect for me too. So that's 
not a blocker; must be some local issye on your end, @ndavis. I think we're 
really close (or ready?) to being able to land this! @bruns, are your comments 
about the question mark a blocker, or are you satisfied?
  
  
  
  
  > I've pinged #syadmin about being unable to apply the patch; hopefully they 
can help us out!
  
  @ndavis - can you upload the `git format-patch -1` output somewhere 
(temporary)?
  
  In D16421#348960 <https://phabricator.kde.org/D16421#348960>, @ngraham wrote:
  
  > In D16421#348959 <https://phabricator.kde.org/D16421#348959>, @bruns wrote:
  >
  > > Tried to reproduce it here, but for me the bottom-right emblem just 
renders fine here:
  > >
  > > F6363939: Unpatched_locked_link_48.png 
<https://phabricator.kde.org/F6363939>
  >
  
  
  There may also be a conceptual issue here - previously, the `emblem-locked` 
which is used for unaccessible (e.g. root owned) files was a lock with orange 
background, now it is green. For locked encrypted files green may be suitable, 
for unacessible files not so much.

REPOSITORY
  R266 Breeze Icons

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

To: ndavis, #vdg
Cc: bruns, ngraham, bcooksley, kde-frameworks-devel, #vdg, michaelh

Reply via email to