-
Notifications
You must be signed in to change notification settings - Fork 104
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update NSView with setShadow:/shadow methods #229
Conversation
I think that this might give some clue as to how to do this in the backend. I am not sure that shadows necessarily need layers to work. |
Also of interest MIGHT be... https://github.com/rcairo/rcairo/blob/master/lib/cairo/context/blur.rb I know these examples are in different languages, but they are both using cairo, so I wanted to note them here. |
It seems as though the keys being used are these...
The new NIBArchive format is proving to be a menace at analyzing nib files for keys... but since we can read xib files it is not a problem, but it is a format we might want to consider looking at in the future. GC |
Yet more info about blur... https://discourse.gnome.org/t/cairo-gaussian-blur-for-shadows/2174 |
Okay, so now I have got everything decoding for the NSShadow from the XIB properly. I am going to work on the backend now. Some changes might be needed in libs-gui, but I doubt it. |
Let me try to explain this once more:
I would suggest to merge this PR here separately and not wait until you have the complete implementation in place. |
Okay. Thank you for the great input. I will change this PR to ready and if you find it suitable, then please approve. |
I will open another PR for the shadow work in GSGraphicsContext. |
@fredkiefer please approve if it's okay to merge. I am in the process of looking at what is involved on the backend. |
#232 PR for set method on libs-gui |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks mostly OK, but the RETAIN issue needs to be resolved. You could also use the setter instead.
I added both retains, I think using the setter is cleaner. I am going to go ahead and make that change. Give me a moment. |
I changed the code to use the setter, thanks. :). Merging. |
Forgot to add the NSView.m file to the commit... I will correct it on master. Sorry. |
Oh... it got them. :). Nevermind. |
Please see the bug here: #217