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.

Sign up for GitHub

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

fix SCT-2914 / PUBLIC-1493 - movement of unselected cells #2076

Open
wants to merge 47 commits into
base: develop
Choose a base branch
from
Open

fix SCT-2914 / PUBLIC-1493 - movement of unselected cells #2076

Show file tree
Hide file tree
Changes from 35 commits
Commits
Show all changes
47 commits
Select commit Hold shift + click to select a range
1b35148
fix SCT-2914 / PUBLIC-1493 - movement of unselected cells
eapearson Jan 14, 2021
6aad8e9
add more test doc
eapearson Jan 14, 2021
632318d
fix code quality issue
eapearson Jan 14, 2021
a4a9a83
Address github code quality alerts
eapearson Jan 14, 2021
5ae04bc
reformat to make codacy happy
eapearson Jan 14, 2021
f10799f
more codacy / stylelint happiness
eapearson Jan 14, 2021
9b60725
more codacy spam
eapearson Jan 14, 2021
2c7e7fa
improve test case property naming (TEST_CASEX -> TEST_CASE_X); move l…
eapearson Jan 21, 2021
f6e3d09
refactor test data to support case and env defaults
eapearson Jan 21, 2021
b63e27c
don't expose service key
eapearson Jan 21, 2021
9539643
use null, not empty string
eapearson Jan 21, 2021
da58646
move some test utils into new NarrativeTesting class (and file), to c…
eapearson Jan 22, 2021
25ef6fe
refactor integration tests
eapearson Jan 22, 2021
2ce09ea
remove inadvertently re-exported functions
eapearson Jan 22, 2021
09e0a65
remove duplicate code, part 1
eapearson Jan 22, 2021
234228e
move test data to separate json file
eapearson Jan 22, 2021
4170676
change test data to match ci changes
eapearson Jan 23, 2021
22454ca
refactor some common code into functions; fix usage of async expect
eapearson Jan 23, 2021
574ef34
test config should use narrativetest as the test user
eapearson Jan 26, 2021
c26d813
use chrome binary in pupeteer
eapearson Jan 26, 2021
c96a1fc
Merge remote-tracking branch 'origin/develop' into fix-PUBLIC-1493
eapearson May 25, 2021
efbd2bd
add unreleased section to release notes
eapearson May 25, 2021
7ec78de
prettier and linting fixes
eapearson May 26, 2021
b46252a
add eslint pre push check (just files that will be pushed).
eapearson May 26, 2021
44a05cf
add auto-creation of docker network for local image run
eapearson May 26, 2021
6c3dbc1
address sonarcloud issue
eapearson May 26, 2021
74c3772
Merge remote-tracking branch 'origin/develop' into fix-PUBLIC-1493
eapearson May 26, 2021
32cf292
fix regressions due to merge
eapearson May 27, 2021
5fe036f
improve eslint prepush
eapearson May 27, 2021
6d3cac4
trivial changes to trigger need testing commit
eapearson May 27, 2021
5f5e644
eslint warnings return error code
eapearson May 27, 2021
4b31bcc
make eslint happy
eapearson May 27, 2021
dff4d92
make sonarcloud happy
eapearson May 27, 2021
ea4d6d9
more pr code quality fixes
eapearson May 27, 2021
11c8619
more pr code quality fixes
eapearson May 27, 2021
f4d2f55
Merge remote-tracking branch 'origin/develop' into fix-PUBLIC-1493
eapearson Jul 27, 2021
02b67d0
Correct linting errors in files merged from develop (?!). WebStorm no…
eapearson Jul 27, 2021
a7dc8e1
Prettify file.
eapearson Jul 27, 2021
1749f9a
resolve linting errors.
eapearson Jul 27, 2021
3e63351
bump and pin npm versions; works, deterministic (nearly).
eapearson Jul 28, 2021
f678d6d
sort scripts
eapearson Jul 28, 2021
b81e4a4
remove unused parameter
eapearson Jul 28, 2021
a6ddfa6
prettier run pre-push (via husky) should only inspect files being pushed
eapearson Jul 28, 2021
68536e5
use puppeteer's chrome binary; otherwise it is too hard to match chro…
eapearson Jul 28, 2021
d5febdd
Merge remote-tracking branch 'origin/develop' into fix-PUBLIC-1493
eapearson Sep 30, 2021
0d7876c
restore data-testid to userMenu (merge regression)
eapearson Sep 30, 2021
820f334
Merge remote-tracking branch 'origin/develop' into fix-PUBLIC-1493
eapearson Nov 22, 2021
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
2 changes: 1 addition & 1 deletion .husky/pre-push
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#!/bin/sh
. "$(dirname "$0")/_/husky.sh"

npm run husky_msg && npm run prettier_check
npm run husky_msg && npm run prettier_check && npm run eslint-pre-push
6 changes: 4 additions & 2 deletions Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,13 @@ build-narrative-container:

docker_image: build-narrative-container

# Per PR #1328, adding an option to skip minification
dev-network:
docker network create kbase-dev || true

dev-image:
SKIP_MINIFY=1 DOCKER_TAG=dev sh $(DOCKER_INSTALLER)

run-dev-image:
run-dev-image: dev-network
ENV=$(ENV) sh scripts/local-dev-run.sh

install:
Expand Down
1 change: 1 addition & 0 deletions RELEASE_NOTES.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ This is built on the Jupyter Notebook v6.0.2 (more notes will follow).
### Unreleased

- SCT-3602 - refseq public data tool now searches by lineage as well; for all public data tools: automatically focus the search input; fix paging bug
- SCT-2914 / PUBLIC-1493 - fix up/down cell movement behavior for unselected cells

### Version 4.4.0
- No ticket: boatloads of code cleanup and fixes to the unit and internal testing
Expand Down
29 changes: 24 additions & 5 deletions kbase-extension/static/kbase/custom/custom.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,16 @@ CSS should go in /kbase-extension/static/kbase/css/kbaseNarrative.css
(or the specific css file if available)
*/
@font-face {
font-family: 'Glyphicons Halflings';
src: url('../../ext_components/bootstrap/dist/fonts/glyphicons-halflings-regular.eot');
src: url('../../ext_components/bootstrap/dist/fonts/glyphicons-halflings-regular.eot?#iefix') format('embedded-opentype'), url('../../ext_components/bootstrap/dist/fonts/glyphicons-halflings-regular.woff2') format('woff2'), url('../../ext_components/bootstrap/dist/fonts/glyphicons-halflings-regular.woff') format('woff'), url('../../ext_components/bootstrap/dist/fonts/glyphicons-halflings-regular.ttf') format('truetype'), url('../../ext_components/bootstrap/dist/fonts/glyphicons-halflings-regular.svg#glyphicons_halflingsregular') format('svg');
font-family: "Glyphicons Halflings";
src: url("../../ext_components/bootstrap/dist/fonts/glyphicons-halflings-regular.eot");
src:
url("../../ext_components/bootstrap/dist/fonts/glyphicons-halflings-regular.eot?#iefix")
format("embedded-opentype"),
url("../../ext_components/bootstrap/dist/fonts/glyphicons-halflings-regular.woff2") format("woff2"),
url("../../ext_components/bootstrap/dist/fonts/glyphicons-halflings-regular.woff") format("woff"),
url("../../ext_components/bootstrap/dist/fonts/glyphicons-halflings-regular.ttf") format("truetype"),
url("../../ext_components/bootstrap/dist/fonts/glyphicons-halflings-regular.svg#glyphicons_halflingsregular")
format("svg");
}

.select2-container--default .select2-results__option--highlighted[aria-selected] {
Expand Down Expand Up @@ -315,12 +322,24 @@ div#notebook {
color: #ccc;
}

.cell.unselected .btn-default:hover {
color: #000;
}

.cell.unselected .kb-cell-toolbar .title-container {
opacity: 0.5;
}

.cell.unselected .kb-cell-toolbar .buttons-container {
opacity: 0.2;
.btn.btn-default.kb-btn-expander.-minimized {
color: rgba(255, 137, 0, 1);
}

.cell.unselected .kb-btn-expander.-minimized {
color: rgba(255, 137, 0, 0.5);
}

.cell.unselected .kb-btn-expander.-minimized:hover {
color: rgba(255, 137, 0, 1);
}

.kb-btn-icon {
Expand Down
105 changes: 53 additions & 52 deletions kbase-extension/static/kbase/js/userMenu.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -57,61 +57,62 @@ define([
}

function render() {
const menu = div({ class: 'dropdown', style: 'display:inline-block' }, [
button(
{
type: 'button',
class: 'btn btn-default dropdown-toggle',
'data-toggle': 'dropdown',
'aria-expanded': 'false',
},
[renderAvatar(), span({ class: 'caret', style: 'margin-left: 5px;' })]
),
ul({ class: 'dropdown-menu', role: 'menu' }, [
li({}, [
a(
{
href: '/#people/' + userName,
target: '_blank',
'data-menu-item': 'userlabel',
},
[
div(
{
style:
'display:inline-block; width: 34px; vertical-align: top;',
},
[
span({
class: 'fa fa-user',
style: 'font-size: 150%; margin-right: 10px;',
}),
]
),
div(
{
style: 'display: inline-block',
'data-element': 'user-label',
},
[displayName, br(), i({}, userName)]
),
]
),
]),
li({ class: 'divider' }),
li({}, [
a({ href: '#', 'data-menu-item': 'logout', id: 'signout-button' }, [
div({ style: 'display: inline-block; width: 34px;' }, [
span({
class: 'fa fa-sign-out',
style: 'font-size: 150%; margin-right: 10px;',
}),
const menu = div(
{ class: 'dropdown', style: 'display:inline-block', dataTestid: 'userMenu' },
[
button(
{
type: 'button',
class: 'btn btn-default dropdown-toggle',
'data-toggle': 'dropdown',
'aria-expanded': 'false',
},
[renderAvatar(), span({ class: 'caret', style: 'margin-left: 5px;' })]
),
ul({ class: 'dropdown-menu', role: 'menu' }, [
li({}, [
a(
{
href: '/#people/' + userName,
target: '_blank',
'data-menu-item': 'userlabel',
},
[
div(
{
style:
'display:inline-block; width: 34px; vertical-align: top;',
Copy link
Collaborator

Choose a reason for hiding this comment

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

can all these inline styles be moved into the relevant stylesheet?

Copy link
Contributor Author

@eapearson eapearson Jul 28, 2021 •
edited
Loading

Choose a reason for hiding this comment

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

Sure, but the goal was not to make as few code changes as possible, whilst adding testing attributes.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, they could be. But the point of changing that file at all was to make a small change to allow it to be testable, not to refactor it.

},
[
span({
class: 'fa fa-user',
style: 'font-size: 150%; margin-right: 10px;',
}),
]
),
div({ style: 'display: inline-block' }, [
span({ dataTestid: 'realname' }, displayName),
br(),
i({ dataTestid: 'username' }, userName),
]),
]
),
]),
li({ class: 'divider' }),
li({}, [
a({ href: '#', 'data-menu-item': 'logout', id: 'signout-button' }, [
div({ style: 'display: inline-block; width: 34px;' }, [
span({
class: 'fa fa-sign-out',
style: 'font-size: 150%; margin-right: 10px;',
}),
]),
'Sign Out',
]),
'Sign Out',
]),
]),
]),
]);
]
);
target.append(menu);
target.find('#signout-button').click(logout);
}
Expand Down
Loading