Fixed toolbar height dynamically - Githubissues

Canvasbird / canvasboard

An interactive board with plugins focusing on replacing the need of multiple platforms for teaching, presenting, managing or explaining your ideas.

MIT License

199 stars 120 forks source link

Fixed toolbar height dynamically #477

Closed Aniket-508 closed 2 years ago

Aniket-508 commented 2 years ago

Issue that this pull request solves

Closes: #472

Brief description of what is fixed or changed

Changed height to max-height property of CSS to 310px

Types of changes

Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[ ] 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
[x] I have commented my code, particularly in hard-to-understand areas
[x] 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

Screenshot (155) Screenshot (156)

Aniket-508 commented 2 years ago

@goliakshay357 Please have a look at this.

goliakshay357 commented 2 years ago

LGTM! Thanks for the enhancement, @Aniket-508!