Skip to content

Conversation

@svdimitr
Copy link
Contributor

This pull request is draft, because the Drawer Icons article has no screenshot, since there is a visual bug.

@svdimitr svdimitr requested a review from marin-bratanov May 20, 2020 15:03
@svdimitr svdimitr self-assigned this May 20, 2020
@svdimitr
Copy link
Contributor Author

Drawer article is updated

@marin-bratanov marin-bratanov marked this pull request as ready for review May 21, 2020 10:31
slug: menu-icons
tags: telerik,blazor,menu,icon,iconclass,image
published: True
position: 2
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Move later in the tree on the left, it is not a very important feature, it should be around templates

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done with fb6a044


You can put an image, icon class or a font icon for each item in the menu to illustrate its purpose for your end users. To apply them, use the following properties:

* for a font icon, populate the `IconField` parameter of the component or provide an `Icon` property in the data model.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fix this up in the same manner as the Drawer article - including factological explanations here, indentations and field names that clearly denote what the fields hold.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done with fb6a044


>caption The result from the code snippet above

![icons](images/icons.jpg)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the image is blurry

@@ -0,0 +1,90 @@
---
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same as the menu article

@marin-bratanov marin-bratanov merged commit eb69b4e into master May 21, 2020
@marin-bratanov marin-bratanov deleted the icon-articles-docs branch May 21, 2020 13:12
marin-bratanov added a commit that referenced this pull request May 21, 2020
* docs(menu): icons article

* chore(menu): updates on icons article

* chore(menu): improvements on icons article

* docs(drawer): added icons article

* chore(drawer): improvements on icons article

* chore(drawer): code snippet improvements

* docs(treeview): added icons article and minor changes to Drawer and Menu icon articles

* chore(docs): improvements on icons articles

* chore(drawer): improvements on icons article

* chore(drawer): updates to Icons articles

* chore(drawer): fix icon article

* chore(menu): Improvements on Icons article

* chore(treeview): improvements on Icons article

* chore(docs): changed icon urls

* chore: icon articles fixes - blurry images, typos, image url

Co-authored-by: Marin Bratanov <m.bratanov@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants