-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
chore: document for svg icon setting #348
Conversation
docs/Basic-Tutorial.md
Outdated
}); | ||
``` | ||
|
||
2. There is a way to use the **actual physical svg file** and **set the file location manually**. |
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.
actual physical svg file
는 그냥 your svg file
정도면 되지 않을까요?
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.
네 정확하게 표현하고 싶어서 작업 하다보니 과했던거 같네요.. 반영하겠습니다.
docs/Basic-Tutorial.md
Outdated
* The disadvantage is that the color must be set by modifying the svg file directly. | ||
* Need to set the path and name for each icon state as shown below. | ||
```js | ||
var instance = new ImageEditor(document.querySelector('#tui-image-editor'), { |
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.
const로 합시다
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.
반영하였습니다. 리뷰 감사합니다.
src/js/ui/theme/standard.js
Outdated
'submenu.normalIcon.color': '#8a8a8a', | ||
'submenu.activeIcon.color': '#e9e9e9', | ||
|
||
// icons (svg icon file setting) |
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.
이 주석들은 기록용으로 남겨둔거예요?
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.
유저한테 다른 형식으로 사용할수 있다는걸 알려주려고 했는데, 어색한것 같네요. 문서에 충분히 설명을 했으니 빼버리겠습니다.
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.
[03/04] 리뷰 완료합니다. 수고하셨습니다.
docs/Basic-Tutorial.md
Outdated
... | ||
}); | ||
``` | ||
* How to get svg file sample |
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.
svg
를 대문자로 쓰면 어떨까요?
-
How to get svg file sample
-
How to get SVG file sample
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.
넵 좋네요 ㅎㅎ svg 쓰인부분 일괄 반영하겠습니다.
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.
문서에 여러 번 등장하는 svg
라는 텍스트는 대문자로 적어주면 어떨까요?
문서를 읽는 사용자들 눈에 잘 띄게요 :)
* feat: default menu svg icon - middle commit * feat: default icon normal Check basic operation * feat: prototype complete for default icon improve * feat: complate default icon code * example file update for default icon setting * chore: svg-loader dependency -> devdependency * chore: document for svg icon setting * fixed some link at document * apply codereview * apply codereview - 2
Please check if the PR fulfills these requirements
fix #xxx[,#xxx]
, where "xxx" is the issue number)Description