-
Notifications
You must be signed in to change notification settings - Fork 241
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/update interface subj dom and some interface features #253
base: main
Are you sure you want to change the base?
Feat/update interface subj dom and some interface features #253
Conversation
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.
Do not merge old branhes into your, we already have this changes. Rebase on develop, please
kb/section_subject_domain_of_ostis_system_interfaces/concepts/concept_data_input_component.scs
Outdated
Show resolved
Hide resolved
...of_ostis_system_interfaces/concepts/concept_data_input_component_without_direct_feedback.scs
Outdated
Show resolved
Hide resolved
kb/section_subject_domain_of_ostis_system_interfaces/concepts/concept_map.scs
Outdated
Show resolved
Hide resolved
kb/section_subject_domain_of_ostis_system_interfaces/NIKA_interface.gwf
Outdated
Show resolved
Hide resolved
kb/section_subject_domain_of_ostis_system_interfaces/NIKA_interface.gwf
Outdated
Show resolved
Hide resolved
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.
- Make real values
- Write possible values for font type, size
- Remove lang_en for numbers. Use class
number
from ims nrel_system_name
is not about text value in UI components. Rename this relation
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.
Update documentation (make UI page) and changelog
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.
Font size is not adaptive. I change it in the KB and see no difference on the interface
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.
After reloading page I see the old values for a second, then they change to the actual from KB
@@ -91,13 +93,60 @@ export const App = () => { | |||
} | |||
} | |||
} | |||
|
|||
|
|||
async function fetchFontFamily() { |
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.
Write what possible values are for font type
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.
Make one relation like concept_ui_component_text_value
and use it. Do not copy it for the copyright panel, menu button and system name. Use only ine relation
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.
agree! Completely agree!
No description provided.