Skip to content
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

Support managing source component in the UI #1624

Merged
merged 1 commit into from
Aug 6, 2018

Conversation

csordasmarton
Copy link
Contributor

Closes #1599

@csordasmarton csordasmarton requested a review from gyorb June 5, 2018 11:22
@csordasmarton csordasmarton force-pushed the ui-component-management branch from ccbf9b1 to 68942e6 Compare June 5, 2018 11:40
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jun 5, 2018
@Ericsson Ericsson deleted a comment Jul 12, 2018
@Ericsson Ericsson deleted a comment Jul 12, 2018
@Ericsson Ericsson deleted a comment Jul 12, 2018
this._componentValue = new SimpleTextarea({
class : 'component-value',
rows : 10,
placeholder : 'Value of the source component. Each line must start '
Copy link
Contributor

Choose a reason for hiding this comment

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

I would recommend a small rephrasing in the placeholder:

with a "+" (results from this path should be listed) or "-" (results from this path should not be listed) sign. E.g.: '+/a/b/x.cpp or -/a/b/'

I would do this rephrasing in the command line and in the user guide too.

@csordasmarton csordasmarton force-pushed the ui-component-management branch from cac1335 to 699828f Compare July 30, 2018 08:34
@csordasmarton csordasmarton requested a review from gyorb July 30, 2018 08:34
@Ericsson Ericsson deleted a comment Jul 30, 2018
Copy link
Contributor

@gyorb gyorb left a comment

Choose a reason for hiding this comment

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

After creating a new component on the UI and going back to select a component for filtering the new component is not listed yet, I need to refresh the filter list or the page.

After saving a new component the cached component names could be refreshed.

@csordasmarton csordasmarton force-pushed the ui-component-management branch from 699828f to a1e7217 Compare August 1, 2018 07:20
@csordasmarton csordasmarton requested a review from gyorb August 1, 2018 07:20
@Ericsson Ericsson deleted a comment Aug 1, 2018
@Ericsson Ericsson deleted a comment Aug 1, 2018
@Ericsson Ericsson deleted a comment Aug 1, 2018
@Ericsson Ericsson deleted a comment Aug 1, 2018
@Ericsson Ericsson deleted a comment Aug 1, 2018
@Ericsson Ericsson deleted a comment Aug 1, 2018
@Ericsson Ericsson deleted a comment Aug 1, 2018
@Ericsson Ericsson deleted a comment Aug 1, 2018
@Ericsson Ericsson deleted a comment Aug 1, 2018
@Ericsson Ericsson deleted a comment Aug 1, 2018
@Ericsson Ericsson deleted a comment Aug 1, 2018
@Ericsson Ericsson deleted a comment Aug 1, 2018
@Ericsson Ericsson deleted a comment Aug 1, 2018
@Ericsson Ericsson deleted a comment Aug 1, 2018
@Ericsson Ericsson deleted a comment Aug 1, 2018
@Ericsson Ericsson deleted a comment Aug 1, 2018
@gyorb gyorb merged commit f28deea into Ericsson:master Aug 6, 2018
@csordasmarton csordasmarton deleted the ui-component-management branch August 14, 2018 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants