WICG / navigation-api

The new navigation API provides a new interface for navigations and session history, with a focus on single-page application navigations.
https://html.spec.whatwg.org/multipage/nav-history-apis.html#navigation-api
486 stars 30 forks source link

Syntax issue in readme #192

Closed fabiancook closed 2 years ago

fabiancook commented 2 years ago

https://github.com/WICG/app-history/blob/7c0332b30746b14863f717404402bc49e497a2b2/README.md?plain=1#L206

Instead of

appHistory.reload({ state: { ...appHistory.current.getState(), test: 3 });

The readme should have:

appHistory.reload({ state: { ...appHistory.current.getState(), test: 3 }});

Seen more clearly:

appHistory.reload({ 
  state: { 
    ...appHistory.current.getState(), 
    test: 3 
  } // <- This is missing here
});

(An additional closing bracket } to close the state object)

domenic commented 2 years ago

Thank you!