Names picked out!

I'm so excited :) we have names picked out. Margaret Eleanor Canon for a girl or Oliver Thomas Canon for a boy. Both have family significance to them. This is a picture of me and my great grandmother Margaret Lee who passed three years ago. We were very close. Also Thomas is a family name my grandad and dad are both Thomas Lee but since my dad had four daughters they didn't get to pass that name down. So we wanted to be able to tie it in somewhere. What do y'all think about them?