<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Hello,<div class=""><br class=""></div><div class=""><blockquote type="cite" class=""><div class="">On 08 May 2015, at 18:35, David Fuhrmann <<a href="mailto:david.fuhrmann@gmail.com" class="">david.fuhrmann@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">Hello Maxime,<br class=""><br class="">Thanks for this cool patch. Does this work with all supported OS X versions (>= 10.6)?<br class=""><br class="">Generally, did you consider only dimming the light when the VLC video is in fullscreen mode? Because in windowed mode, the rest of the monitor will probably so bright that it does not really make sense to dim the keyboard (and maybe the user wants to use it still).<br class=""><br class="">Please also see some minor remarks about your patch inline.</div></blockquote><div class=""><br class=""></div><div class="">Thanks for the comments. It should work, the IOKit methods used are supported since 10.5.</div><div class=""><br class=""></div><div class="">Indeed, it makes sense. Actually I got the idea while watching a video in fullscreen !</div><div class=""><br class=""></div><div class="">I’ll make the necessary changes. By the way, what is the proper way to send an edited patch to the mailing list ? Should I use git send-email with the same commit log text ?</div><div class=""><div apple-content-edited="true" class="">
<div style="color: rgb(0, 0, 0); letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div style="color: rgb(0, 0, 0); letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div style="color: rgb(0, 0, 0); letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div style="color: rgb(0, 0, 0); letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div style="color: rgb(0, 0, 0); letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><br class="Apple-interchange-newline"><div>Regards,</div><div>Maxime,</div></div></div></div></div></div></div></div><br class=""></div></div></body></html>