🦋 Changeset detected

Latest commit:4ed7cda

The changes in this PR will be included in the next version bump.

Not sure what this means?Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Codecov Report

Attention: Patch coverage is25.00000% with12 lines in your changes missing coverage. Please review.

Project coverage is 65.19%. Comparing base(068e23c) to head(4ed7cda).

@@               Coverage Diff               @@##           graphiql-v4    #3753      +/-   ##===============================================- Coverage        65.30%   65.19%   -0.12%===============================================  Files              121      121                Lines             7006     7021      +15       Branches          2263     2265       +2     ===============================================+ Hits              4575     4577       +2- Misses            2414     2427      +13  Partials            17       17

I see it's still WIP so maybe persistence is already planned, i think we should provide hooks for handling and setting tab name (so for example explorer could change the tab name), but i want to improve the hooks design. we already provide hooks for controlling tabs. we should provide tabs for reading and controlling tab state, including tab name. ideally we could combine some of this into something likeuseTabState(), perhaps this can be the beginning of that? this is also important for projects that need to persist the full editor state remotely via the storage API override, so either way it's ideal to persist this state and perhaps make it manageable via context

there was something else I wanted to reccomend here but i forgot, I'll come back to this, but thanks for all the initiative!