Skip to content
This repository has been archived by the owner on Nov 19, 2020. It is now read-only.

Implement context menu for record nav (fix #65) #67

Merged
merged 1 commit into from Aug 29, 2016

Conversation

harunurhan
Copy link
Collaborator

  • Implement tree-menu and tree-menu-item components.
  • Implement window-href service that gets window.location.href.
  • Set value path as an id to each component in order to navigate via #.
  • Introduce shared dom-utils service with focus, select and flash utilities.
  • Focus the first input child when navigated to a component.
  • Implement shallow search for menu and navigation by typing.

GIFS

You can search things, and navigate to them by enter if you give the exact path.

type-tree-menu

Clicking things works too.

click-tree-menu

* Implement tree-menu and tree-menu-item components.

* Implement window-href service that gets window.location.href.

* Set value path as an id to each component in order to navigate via #.

* Introduce shared dom-utils service with focus, select and flash utilities.

* Focus the first input child when navigated to a component.

* Implement shallow search for menu and navigation by typing.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 90.96% when pulling 4149813 on harunurhan:nav-menu into 8b93466 on inspirehep:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 90.96% when pulling 4149813 on harunurhan:nav-menu into 8b93466 on inspirehep:master.

}

.flash {
border: 2px solid yellow;
}
Copy link
Contributor

Choose a reason for hiding this comment

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

I think you should configure your editor to add newlines at the end of files :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It will be in another pr where I fix tslint errors (which also forces to add new-line at the end of *.ts files.)

@kaplun kaplun merged commit 7b3ddb0 into inspirehep:master Aug 29, 2016
@harunurhan harunurhan deleted the nav-menu branch January 17, 2018 12:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants