canvasboard icon indicating copy to clipboard operation
canvasboard copied to clipboard

Updated the daily quote section - added padding, colour and right-quote img

Open vismitap opened this issue 3 years ago β€’ 8 comments

Fixed #373

Issue that this pull request solves

Closes: #373

Proposed changes

The proposed changes were:

  • The daily quote should appear on the development server
  • The quote should have a good bottom padding
  • The quote text color should be a bit nicer and attractive

Brief description of what is fixed or changed

Changes made are:

  • Added right quote image to give the quote a better and complete look. Earlier there was just on img <img src="https://img.icons8.com/fluent-systems-filled/24/000000/quote-left.png"/>, so closed it with <img src="https://img.icons8.com/fluent-systems-filled/24/000000/quote-right.png"/>.
  • Changed the opacity and colors of both quoteText and quoteAuthor to make it look more attractive.
  • Added padding at the bottom of quote section.

Types of changes

Put an x in the boxes that apply

  • [x] Bugfix (non-breaking change which fixes an issue)
  • [x] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [ ] Documentation update (Documentation content changed)
  • [ ] Other (please describe):

Checklist

Put an x in the boxes that apply

  • [x] My code follows the style guidelines of this project
  • [x] I have performed a self-review of my own code
  • [ ] I have commented my code, particularly in hard-to-understand areas
  • [ ] I have made corresponding changes to the documentation
  • [x] My changes generate no new warnings
  • [x] My changes does not break the current system and it passes all the current test cases.

Screenshots

Please attach the screenshots of the changes made in case of change in user interface

Final output looks like this: image

vismitap avatar Feb 24 '21 21:02 vismitap

Hello there!πŸ‘‹ Welcome to the project!πŸ’–

Thank you and congratsπŸŽ‰ for opening your first pull request.✨ Canvasboard is an interactive board with plugins focusing on replacing the need of multiple platforms for teaching, presenting or explaining your ideas. The interactive plugins are built using web technologies to ease your work right within a single platform. Please adhere to our Code of Conduct.πŸ™Œ We will get back to you as soon as we can πŸ˜„. Feel free to join our Slack Community.πŸ’– We have different channels for active discussions.✨ Hope you have a great time there!πŸ˜„

welcome[bot] avatar Feb 24 '21 21:02 welcome[bot]

Hey @vismitap, Thanks for your interest for solving this issue but I don't see the quotes when workspace are filled. Screenshot image

Couple of suggestions:

  1. Following the color format would be great. (Same patterns of black is preferred)
  2. Quote shall always be in the bottom of the page, even if the workspace is empty.

Please feel free to message us on slack channel in case if you need any help or guidance. Thanks

goliakshay357 avatar Feb 25 '21 13:02 goliakshay357

Hey @vismitap, Thanks for your interest for solving this issue but I don't see the quotes when workspace are filled. Screenshot image

Couple of suggestions:

  1. Following the color format would be great. (Same patterns of black is preferred)
  2. Quote shall always be in the bottom of the page, even if the workspace is empty.

Please feel free to message us on slack channel in case if you need any help or guidance. Thanks

  • I changed the colour because that was mentioned in the deliverable in #373, I can change that to black again if you want.
  • I can see the quote even after adding the workspace in the local host. image
  • The quote is always in the bottom as you can see in the screenshot. I don't know what the issue is in the deployment. πŸ€”

Hey btw, how are you checking in deployment server even before the code is merged? Is there any tool for that? πŸ€”πŸ€©

vismitap avatar Feb 25 '21 14:02 vismitap

Hey @vismitap, Netlify provides us the flexibility of auto deployment preview of each PR. Can you message me on slack? Would love to debug together.

Thanks

goliakshay357 avatar Feb 25 '21 14:02 goliakshay357

Hey @vismitap, Netlify provides us the flexibility of auto deployment preview of each PR. Can you message me on slack? Would love to debug together.

Thanks

Yes sure!

vismitap avatar Feb 25 '21 14:02 vismitap

What's your username on slack? Could not find you on canvas board channel.

vismitap avatar Feb 25 '21 14:02 vismitap

What's your username on slack? Could not find you on canvas board channel.

Display name: Goli Akshay Sujith Slack Invite Thanks

goliakshay357 avatar Feb 25 '21 14:02 goliakshay357

Hey @goliakshay357 ! Added json file having a list of quotes in the src/interfaces folder and changed the color of quotes back to black. Added some padding to the container as well. I hope it works now! Here is the screenshot: image

vismitap avatar Feb 27 '21 18:02 vismitap