It would make sense to mention why we need this. Also, this should have been the first element in the enum, is it too late for that? If it is, do you want to assign some value to it to leave a gap in case you need other chrome values? (i.e.
mir_shell_chrome_normal, mir_shell_chrome_low, mir_shell_chrome_none = 1000
or some such thing?)
« Back to merge proposal
It would make sense to mention why we need this. Also, this should have been the first element in the enum, is it too late for that? If it is, do you want to assign some value to it to leave a gap in case you need other chrome values? (i.e.
mir_shell_ chrome_ normal, chrome_ low, chrome_ none = 1000
mir_shell_
mir_shell_
or some such thing?)