'High School Musical' star Ashley Tisdale engaged to Christopher French
Ashley Tisdale, a cast member in "Scary Movie V," poses with boyfriend Christopher French at the Los Angeles premiere of the film at the Cinerama Dome in Los Angeles on Thursday, April 11, 2013. (Chris Pizzello / Invision)
Published Friday, August 9, 2013 3:33PM EDT
Ashley Tisdale is engaged to Christopher French.
The 'High School Musical' actress was overjoyed after Christopher, who she first went public with in December 2012, popped the question on the top of the Empire State Building in New York on Thursday night.
She tweeted: "Best night of my life. We're on top of the world."
While friends say Ashley wasn't surprised by the proposal, she was thrilled by Christopher's choice of location.
A source told Us Weekly: "He had been planing on it for awhile. I think she knew it was coming but she is so happy.”
Ashley, 28 - who has previously dated musician Jared Murillo and music video director Scott Speer – has revealed she likes "mysterious bad boys".
She said: "There's definitely a thing where I like the dark, mysterious bad boy."
Ashley prefers an "old-school" approach to dating and would rather be honest about her feelings than play games with guys she likes.
She said: "I'm definitely old-school when it comes to dating.
"I'm not into the 'game' so much. If I like you, I'll confront you and be open about it. Then I expect you to come after me."
The actress has previously admitted she loves giving dating advice to people.
She has said: "I give dating advice on a regular basis. It's not that I'm any expert, but it's always nice to share that with your friends."
Ashley's biggest relationship tip relates to the importance of communication.
She explained: "Communication is so key. Especially in this business, I'm always travelling and doing something else. So I think communication is the foundation of it all."