[Dashboard] When editing visualization with session storage on, compressed URL is lost

[majagrubic]

Kibana version:
7.9 BC2

Original install method (e.g. download page, yum, from source, etc.):
Cloud

Describe the bug:
I believe the root cause is the same as #73013, but putting it here just in case it isn't.
When editing a visualization that was added through an "Add option", if state:storeInSessionStorage advanced setting is turned on, the URL defaults to a long one.

Steps to reproduce:

1. Go to Advanced settings
2. Turn on state:storeInSessionStorage setting
3. Reload the page; make sure the setting is applied
4. Select "Add" and add an existing visualization
5. Edit existing visualization
6. Observe the changed URL

Expected behavior:
URL stays compressed

Screenshots (if relevant):

Errors in browser console (if relevant):

Provide logs and/or server output (if relevant):

Any additional context:

[elasticmachine]

Pinging @elastic/kibana-app (Team:KibanaApp)

[majagrubic]

Also, when sharing a dashboard via permalink a regular URL is generated.

[Dosant]

Also, when sharing a dashboard via permalink a regular URL is generated.

afaik, this is expected.
Otherwise sharing won't makes sense because state won't be stored in the url.

[flash1293]

True, @Dosant . Also, short url should handle it gracefully (without putting everything in the URL at any point)

[majagrubic]

Sorry, I am on vacation this week and away from computer, but I am pretty sure I checked this on 7.8 and it worked differently. Would either one of you mind verifying this and then closing it if it's not an issue? 🙏 @flash1293 @Dosant

[majagrubic]

Ah sorry, for permalinks you are right - it is indeed the expected behavior.
But the original issue of losing compressed URL when editing visualization isn't.

[kmartastic]

@ThomThomson you have a sense of LOE on this?

[ThomThomson]

I am unable to recreate this in Master, the compressed URL is retained throughout adding panels, editing panels etc. Will close for now, and we can reopen if this issue crops up again.