-
Notifications
You must be signed in to change notification settings - Fork 9
Text updates - mainly readme.md #13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you were updating the Read Me could you please change the images to reflect how the block now appears please?
Change log | ||
===================== | ||
Please Look in CHANGELOG | ||
Please look in CHANGELOG |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please look in CHANGELOG | |
Please look in CHANGELOG. |
return html_writer::div($image, 'image_wrap'); | ||
} else { | ||
// We need a CSS soloution apparently lets give it to em. | ||
// We need a CSS solution apparently lets give it to 'em. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// We need a CSS solution apparently lets give it to 'em. | |
// We need a CSS solution apparently. Lets give it to 'em. |
Or X gon' give it to ya.
} | ||
|
||
// Make sure user is still enroled. | ||
// Make sure user is still enrolled. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// Make sure user is still enrolled. | |
// Make sure the user is still enrolled. |
Mainly updated some of the text/comments here and there.