-
Notifications
You must be signed in to change notification settings - Fork 23
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
feat(stark-ui): implementation of minimap component #824
feat(stark-ui): implementation of minimap component #824
Conversation
export const translationsEn: object = { | ||
STARK: { | ||
TABLE: { | ||
TOGGLE_COLUMNS: "Column filters" |
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.
Could you please move this translation in stark-core/src/common/translations/translations ? 😊
Like this all the translations are in the same location
2026fff
to
fa15c2e
Compare
/* ============================================================================== */ | ||
/* S t a r k M i n i m a p */ | ||
/* ============================================================================== */ | ||
/* stark-ui: src/modules/minimap/components/minimap-theme.scss */ |
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.
The theme.scss
file should contain only "themable" properties, meaning that only colors, backgrounds, fill colors, foregrounds, etc should be defined in this file
So let's move all the other CSS rules like top, left, right, etc to the _minimap.component.scss
file ;)
fa15c2e
to
c972fc3
Compare
/* ============================================================================== */ | ||
/* S t a r k M i n i m a p */ | ||
/* ============================================================================== */ | ||
/* stark-ui: src/modules/minimap/components/minimap-theme.scss */ |
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.
c972fc3
to
9965ae8
Compare
9965ae8
to
4148d7d
Compare
ISSUES CLOSED: #758
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
Issue Number: #758
What is the new behavior?
The minimap composent is now implemented
Does this PR introduce a breaking change?
Other information