For simply content scrapping without javascript and ajax content try scrapy for best practices. Easy tutorial to learn scrapy ScrapingHub Tutorial Official site of ScrappingHub
Selenium is good for scraping dynamic content and causes unnecessary overhead as mentioned in above answer.
For above code:
Try avoiding time.sleep and use EC.presence_of_element_located and similar functions to obtain desired behavior. Selenium Waits