query-monitor icon indicating copy to clipboard operation
query-monitor copied to clipboard

Inline scripts and styles

Open szepeviktor opened this issue 9 years ago • 15 comments

Could you mark localized scripts (data) and inline styles (extra) in Scripts and Styles?

szepeviktor avatar Mar 22 '15 23:03 szepeviktor

Yeah I probably could. Thanks for the suggestion.

johnbillion avatar Mar 23 '15 00:03 johnbillion

I've been working on implementing this, and I'm getting close:

screen shot 2016-07-22 at 2 44 07 pm

but can't determine the best way to handle large amounts of inline data:

screen shot 2016-07-22 at 2 45 27 pm

@johnbillion do you think simply limiting the output to a certain number of characters is a decent solution?

crstauf avatar Jul 22 '16 18:07 crstauf

Maybe get the first line and truncate it at 80 or 100 ms_strlen().

szepeviktor avatar Jul 22 '16 18:07 szepeviktor

I'm wondering, if it's even worth it to show the inline data... it's in the source of the page, so perhaps just indicating that there is inline data is sufficient (as if you're interested, you could go look find it in the source)?

crstauf avatar Jul 22 '16 18:07 crstauf

hmmm yeah I think providing the first 60 characters or so would be enough; user could grab the characters and search for them in the source. will do that.

crstauf avatar Jul 22 '16 18:07 crstauf

I'm wondering, if it's even worth it to show the inline data...

Yes, my issue was: "Could you mark ..."

szepeviktor avatar Jul 22 '16 18:07 szepeviktor

yeah, that's what got me thinking about even showing it... I do and don't like showing it; seems redundant, but also helpful for a quick glance. thinking it over

crstauf avatar Jul 22 '16 19:07 crstauf

thoughts, everyone?

screen shot 2016-07-22 at 3 08 59 pm

crstauf avatar Jul 22 '16 19:07 crstauf

Could you add the count? data and 2×after

szepeviktor avatar Jul 22 '16 19:07 szepeviktor

@szepeviktor count how many items are in the data and after keys? data is only a string, but after can have multiple items... is that what you're wanting? a count of how many inline scripts have been added?

crstauf avatar Jul 22 '16 19:07 crstauf

@szepeviktor how's this?

screen shot 2016-07-22 at 4 06 40 pm

crstauf avatar Jul 22 '16 20:07 crstauf

OK.

szepeviktor avatar Jul 22 '16 20:07 szepeviktor

@johnbillion Is there something in particular you're wanting to do with this, that kept it from going through in 2016? Please let me know; happy to do what needs to be done to get this into QM core.

crstauf avatar Feb 21 '19 06:02 crstauf

I think I decided that I wanted to expose more information than this, or in a better way, but I ended up having decision paralysis. I'll check out your updated PR, thanks.

johnbillion avatar Feb 21 '19 10:02 johnbillion

decision paralysis

I know what that is!

szepeviktor avatar Feb 21 '19 11:02 szepeviktor