black-studio / black-studio-tinymce-widget

Black Studio TinyMCE Widget plugin for WordPress
https://wordpress.org/plugins/black-studio-tinymce-widget/
GNU General Public License v3.0
22 stars 13 forks source link

Refactor #2

Closed cfoellmann closed 10 years ago

cfoellmann commented 10 years ago

Suggestions for refactoring of the codebase:

cfoellmann commented 10 years ago

@marcochiesi what is your plan? I would love to start with this but do not want to do major changes while you are doing it in parallel.

Urgent: Due to the added .git* files the files need to be normalized! You should see (in your client (I can recommend http://www.sourcetreeapp.com/)) that the files have changed completely without the code changing. Just commit all file changes, please.

marcochiesi commented 10 years ago

@cfoellmann The plan is to refactor the codebase using an OOP-oriented approach and provide a better file/folder organization, exactly as you suggested.

As for the file normalization, it should be ok now. Please check.

cfoellmann commented 10 years ago

looking good! I would suggest creating a new branch refactor based on develop to start the refactoring. Maybe we can have a short text chat over skype (same username as here on GitHub) to set some targets for this. Ping me there to start a short dialogue.

Would love to see a first version this week :imp: - Want to make some free time for this

cfoellmann commented 10 years ago

Great stuff you did on the refactoring!

cfoellmann commented 10 years ago

Lets close this and proceed by opening new issues with specific problems/improvements

cfoellmann commented 10 years ago

But reference this issue #2 in other connected items