canvasboard icon indicating copy to clipboard operation
canvasboard copied to clipboard

Removed the navigate to top button

Open sakkshm26 opened this issue 3 years ago β€’ 4 comments

Issue that this pull request solves

Closes: #455

Proposed changes

Brief description of what is fixed or changed

Removed the navigate to top button

Types of changes

Put an x in the boxes that apply

  • [ ] Bugfix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [x] 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
  • [ ] My changes does not break the current system and it passes all the current test cases.

Screenshots

Screenshot (400)

Other information

Any other information that is important to this pull request

sakkshm26 avatar Oct 23 '21 19:10 sakkshm26

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 Oct 23 '21 19:10 welcome[bot]

@goliakshay357 can you please review this PR?

sakkshm26 avatar Oct 29 '21 17:10 sakkshm26

Hello @sakkshm26, Please check the review comment. As mentioned, You cannot delete the global component.

One way to solve this problem is by manually injecting the component in the required templates instead of just adding it in the app-component [which is the main component].

Let me know if you need any help. Thanks :)

goliakshay357 avatar Oct 29 '21 17:10 goliakshay357

@goliakshay357 Is scroll-to-top the required template?

sakkshm26 avatar Oct 30 '21 07:10 sakkshm26