Skip to content

Search UI - #4716

Closed
kivinen wants to merge 1 commit into
ietf-tools:feat/tzawarefrom
kivinen:search-ui
Closed

Search UI#4716
kivinen wants to merge 1 commit into
ietf-tools:feat/tzawarefrom
kivinen:search-ui

Conversation

@kivinen

@kivinen kivinen commented Nov 6, 2022

Copy link
Copy Markdown
Contributor

Implements #4711 i.e., allows using control-click or context menu to open items in the top right search list to open multiple documents in different tabs.

@codecov

codecov Bot commented Nov 6, 2022

Copy link
Copy Markdown

Codecov Report

❗ No coverage uploaded for pull request base (feat/tzaware@fbd9509). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head 10337a7 differs from pull request most recent head 4ec3a9b. Consider uploading reports for the commit 4ec3a9b to get more accurate results

@@               Coverage Diff               @@
##             feat/tzaware    #4716   +/-   ##
===============================================
  Coverage                ?   88.51%           
===============================================
  Files                   ?      296           
  Lines                   ?    39840           
  Branches                ?        0           
===============================================
  Hits                    ?    35263           
  Misses                  ?     4577           
  Partials                ?        0           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@larseggert

Copy link
Copy Markdown
Collaborator

There are some meeting-related changes in this PR that seem unrelated?

@kivinen

kivinen commented Nov 7, 2022 via email

Copy link
Copy Markdown
Contributor Author

@larseggert

Copy link
Copy Markdown
Collaborator

You need to create your new branch off tzaware, not of another branch that has changes.

@rjsparks

rjsparks commented Nov 9, 2022

Copy link
Copy Markdown
Member

At this point, it would be better to recreate this PR from current main.
I took a look at detangling this myself, and I could do so in a pinch, but I think it will be easier for Tero when he has a moment.

@kivinen

kivinen commented Nov 9, 2022 via email

Copy link
Copy Markdown
Contributor Author

@rjsparks

Copy link
Copy Markdown
Member

This is moved to #4747.

@rjsparks rjsparks closed this Nov 10, 2022
@rjsparks

Copy link
Copy Markdown
Member

(I tried to do a soft-reset juggle, but ended up with exactly the opposite of what I wanted. I built the correct thing on a fresh checkout of main instead.)

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Nov 14, 2022
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.

3 participants