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

Move filter to query param and highlight filter matches on graphs #310

Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
58 commits
Select commit Hold shift + click to select a range
bdc0220
Add a TraceGraph view (#273) (#276)
copa2 Dec 18, 2018
4112482
Revisions for search and trace detail embed mode (#286)
tiffon Dec 20, 2018
2baf075
Unfocus unit test (#298)
tiffon Dec 20, 2018
d90d042
Add a Button to Reset Viewing Layer Zoom (#215) (#290)
everett980 Jan 3, 2019
29b340d
Add a copy icon to entries in KeyValuesTable (#204) (#292)
everett980 Jan 4, 2019
e0f34e4
Add popover and prevent submit if duration params are invalid (#244) …
everett980 Jan 4, 2019
bf70c14
Add indent guides to trace timeline view (#172) (#297)
everett980 Jan 4, 2019
07c7f4f
Add ability to search for nodes in TraceDiffGraph and TraceGraph (#307)
everett980 Jan 7, 2019
1953db5
Style GraphSearch, Debounce query param updating
everett980 Jan 8, 2019
e918deb
Use hex instead of html colors, remove commented code
everett980 Jan 8, 2019
181562c
Handle falsy graphSearch value in state, improve typing
everett980 Jan 8, 2019
edf4805
Move GraphSearch and use as utility
everett980 Jan 8, 2019
d1e05d6
Add tests for GraphSearch
everett980 Jan 8, 2019
6ea7d6a
Update create-react-app to 2.1.2 (#302)
tiffon Jan 9, 2019
5fa907b
Add tests for consuming GraphSearch, Add tests for filter-spans
everett980 Jan 9, 2019
0122fd7
Improve spy variable name
everett980 Jan 9, 2019
f08a609
Ability to open additional menu links in same tab (Resolves #275) (#278)
zablvit Jan 9, 2019
8e2df85
Revive the changelog (#300)
tiffon Jan 12, 2019
7ea05f4
Add SpanID support to filter-spans, Consolidate textFilter & GraphSearch
everett980 Jan 15, 2019
2bf1b43
Unify uiFind name, fix refs, improve types, remove commented code
everett980 Jan 15, 2019
353d69c
Add tests to new & tested files, merge common/GraphSearch into user
everett980 Jan 15, 2019
99b7a6c
Update exisitng test files
everett980 Jan 16, 2019
10b5194
Polish filter-spans
everett980 Jan 16, 2019
dd9f14c
Merge branch 'jaegertracingmaster' into issue-307-add-node-search-to-…
everett980 Jan 16, 2019
0c5b591
Use uppercase types, remove constructor for initial state
everett980 Jan 16, 2019
50db264
Update enzyme, Re-enable and add TracePage/index.test.js tests
everett980 Jan 25, 2019
9a7e0c1
Merge branch 'jaegertracingmaster' into issue-307-add-node-search-to-…
everett980 Jan 25, 2019
ef880c7
Merge branch 'issue-307-add-node-search-to-tracegraph-and-tracediffgr…
everett980 Jan 25, 2019
e531d6e
Add tests to TraceDiff/
everett980 Feb 6, 2019
fe420c6
Treat all span relationships equally, make ui-find-match css more obv…
everett980 Feb 7, 2019
a5178e0
Fix node/edge layering, simplify ui-find-match css
everett980 Feb 7, 2019
66f080d
Change diffNode ui match color, scale ui match outline size
everett980 Feb 15, 2019
9dc1a4e
Calculate box-shadow size on nodesContainer not drawNode
everett980 Feb 20, 2019
f685d47
Fix color attributes to preserve node text color
everett980 Feb 21, 2019
b6b40c0
Rename UIFind to UiFind, Update TraceDiffFind
everett980 Feb 21, 2019
acd3e66
Finish DiffGraph find css, Render 0 count suffix
everett980 Feb 25, 2019
2434056
merge master and update test
everett980 Feb 25, 2019
51337fb
Fix git case insensitivity issue
everett980 Feb 26, 2019
4313988
Merge branch 'issue-307-add-node-search-to-tracegraph-and-tracediffgr…
everett980 Feb 26, 2019
1f5421d
Merge updated graph find, maintain increased test coverage
everett980 Feb 26, 2019
69a576a
Remove redundant find, Fix TraceGraph suffix, Fix flow complaint, WIP
everett980 Mar 1, 2019
259cd13
Clean up WIP commit
everett980 Mar 5, 2019
15c51d5
Clean up CSS, Track only TracePage, Fix span-ancestor-ids logic
everett980 Mar 5, 2019
a1f5e59
Ensure scroll to bottom scrolls to last visible span
everett980 Mar 5, 2019
aa3f2e5
Move addToUiFind logic to VirtualizedTraceView
everett980 Mar 5, 2019
df9bafd
Merge branch 'master' into issue-307-add-node-search-to-tracegraph-an…
everett980 Mar 5, 2019
7422720
Merge branch 'issue-307-add-node-search-to-tracegraph-and-tracediffgr…
everett980 Mar 6, 2019
c5034fa
Process FOLLOWS_FROM spans in TraceView (#335)
rubenvp8510 Mar 7, 2019
95026f9
Move graph utils to new file, Clean up span-ancestor-id loop
everett980 Mar 7, 2019
c3d4d14
fix merge again
everett980 Mar 8, 2019
bf3386f
Merge branch 'issue-307-add-node-search-to-tracegraph-and-tracediffgr…
everett980 Mar 8, 2019
b4afe2d
Update tests after merge
everett980 Mar 8, 2019
de6571f
Merge pull request #1 from everett980/issue-239-fix-TracePageHeader-t…
everett980 Mar 8, 2019
5386868
Clean up and DRY out tests, Change trackFilter to function
everett980 Mar 11, 2019
5971072
Merge branch 'jaegertracingmaster' into issue-307-add-node-search-to-…
everett980 Mar 11, 2019
c5f40c8
Fix yarn.lock urls
everett980 Mar 11, 2019
8a4b0fb
Fix yarn.lock change %2f back to /
everett980 Mar 11, 2019
d3965dc
Merge branch 'master' into issue-307-add-node-search-to-tracegraph-an…
tiffon Mar 12, 2019
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .flowconfig
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
./flow-typed/npm

[options]
suppress_comment= \\(.\\|\n\\)*\\$FlowIgnore

[version]
0.71.0
4 changes: 2 additions & 2 deletions packages/jaeger-ui/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,8 @@
"babel-plugin-import": "1.11.0",
"bluebird": "^3.5.0",
"customize-cra": "0.2.9",
"enzyme": "^3.2.0",
"enzyme-adapter-react-16": "^1.1.0",
"enzyme": "^3.8.0",
"enzyme-adapter-react-16": "^1.2.0",
"enzyme-to-json": "^3.3.0",
"http-proxy-middleware": "^0.19.1",
"less": "3.9.0",
Expand Down
16 changes: 8 additions & 8 deletions packages/jaeger-ui/src/components/TraceDiff/TraceDiff.js
Original file line number Diff line number Diff line change
Expand Up @@ -48,22 +48,22 @@ type State = {
graphTopOffset: number,
};

function syncStates(urlSt, reduxSt, forceState) {
const { a: urlA, b: urlB } = urlSt;
const { a: reduxA, b: reduxB } = reduxSt;
function syncStates(urlValues, reduxValues, forceState) {
const { a: urlA, b: urlB } = urlValues;
const { a: reduxA, b: reduxB } = reduxValues;
if (urlA !== reduxA || urlB !== reduxB) {
forceState(urlSt);
forceState(urlValues);
return;
}
const urlCohort = new Set(urlSt.cohort || []);
const reduxCohort = new Set(reduxSt.cohort || []);
const urlCohort = new Set(urlValues.cohort);
const reduxCohort = new Set(reduxValues.cohort || []);
if (urlCohort.size !== reduxCohort.size) {
forceState(urlSt);
forceState(urlValues);
return;
}
const needSync = Array.from(urlCohort).some(id => !reduxCohort.has(id));
if (needSync) {
forceState(urlSt);
forceState(urlValues);
}
}

Expand Down
Loading