Fix #627: Clear fields in talkDetailsWidget in Talk Editor when removing a talk

Freeseer / freeseer

Designed for capturing presentations at conferences. Pre-fill a list of talks to record, record them, and upload them to YouTube with our YouTube Uploader.

http://freeseer.readthedocs.org

GNU General Public License v3.0

215 stars 107 forks source link

Fix #627: Clear fields in talkDetailsWidget in Talk Editor when removing a talk #672

Closed benbuckley closed 10 years ago

benbuckley commented 10 years ago

I noticed issue #627 and thought I would take a shot at fixing it.

EDIT: I just noticed the issue number was 627 and this pull request is 672. That makes me oddly happy.

dideler commented 10 years ago

Before I can merge, the fields should also be cleared when "Remove All" is pressed.

It's good practice to reference the issue from the commit, in this case prepend "Fix #627" to the commit message.

benbuckley commented 10 years ago

How is it now, @dideler ?

dideler commented 10 years ago

Looks good, thanks!