Feat: Add Undo and Redo functionality

[ravjot07]

Add: Undo and Redo for pen stoke..

Undo-Redo.mp4

[quozl]

Thanks.

• your commit messages do not follow our guidance, in particular
  • "added" should be "add",
  • the first lines are too long,
  • there's nothing but first lines,
  • "implemented" is vague,
  • typo "stoke",
• you have missing newline on files,
• the undo and redo icons do not follow Sugar style, please use our existing icons for undo and redo; remember each activity is just one of several, so don't focus only on this repository.

@walterbender, please review? (You posted on issue #14).

[ravjot07]

Thanks.

• your commit messages do not follow our guidance, in particular

  • "added" should be "add",
  • the first lines are too long,
  • there's nothing but first lines,
  • "implemented" is vague,
  • typo "stoke",

• you have missing newline on files,

• the undo and redo icons do not follow Sugar style, please use our existing icons for undo and redo; remember each activity is just one of several, so don't focus only on this repository.

@walterbender, please review? (You posted on issue #14).

@quozl can you review this undo and redo button styles and i have resolved most of the suggestions as you mentioned sir

[quozl]

You can find edit-undo.svg in sugar-artwork repository, icons/scalable/actions

[ravjot07]

sugar-artwork

@quozl Undo and Redo button Updated according to sugar-artwork

[quozl]

Thanks. @chimosky, @walterbender any review comments?

[chimosky]

Your commit message says "Implemented a stoke history data structure", I'm guessing you meant "stroke history".

If that's the case then you should edit the commit message.

[chimosky]

Reviewed, not tested.

I'm genuinely curios why you add lots of spaces in most of your commits, a commit message says one thing and after looking at the commit most of the changes are new line additions which is just noise.

[ravjot07]

Reviewed, not tested.

I'm genuinely curios why you add lots of spaces in most of your commits, a commit message says one thing and after looking at the commit most of the changes are new line additions which is just noise.

@chimosky This could be due to the linting tools configured in my setup, which might be automatically adjusting whitespace or adding new lines. I’ll review my configuration to ensure cleaner commits going forward.

[ravjot07]

@quozl @chimosky

[ravjot07]

@quozl if there are no further changes we should merge this pr

[quozl]

#26 (comment) is my review of commit messages, no changes have happened yet.