profile
viewpoint
Jon Bell jonbell-lot23 Wellington, NZ http://lot23.com Jon Bell

jonbell-lot23/Rodin 1

A fork of hypertextual so I can learn how forking and merging work

jonbell-lot23/tilde.club 1

Code and documentation for setting up and managing a tilde.club server

jonbell-lot23/bosch 0

A fork of a fork of a fork of Notational Velocity

jonbell-lot23/contentful-test-1 0

Gatsby starter for a Contentful project from the community.

jonbell-lot23/cosmic-test-1 0

🚀⚡️ Blazing fast blog built with Gatsby and the Cosmic Headless CMS 🔥

jonbell-lot23/devtools 0

Web app with developer tools built on the Record Replay Protocol

jonbell-lot23/enthusiasticpanther 0

A make-believe band with real stats

issue openedRecordReplay/devtools

Put Loom link behind experimental flag

Our Loom functionality doesn't look particularly ready for prime time yet. We originally pushed it out to gather information, but there's a branding overhead to having something feel half-baked in the UI like this. Everything a new user sees needs to feel polished and relevant.

image

I recommend putting it behind an experimental flag while we work out what to do with this feature idea.

created time in a day

issue commentRecordReplay/devtools

Implement an indeterminate loading UI

Here's the privacy+loading ticket I mentioned, which is more important than the animation bit: https://github.com/RecordReplay/devtools/issues/4541

jaril

comment created time in a day

issue commentRecordReplay/devtools

Implement an indeterminate loading UI

And so this doesn't got lost in the shuffle, note that we also have the privacy loading screen revision queued up and ready to go.

So even if we don't have the animation bit ready to go, we can at least address the concerns we have about the current privacy/loader.

jaril

comment created time in 2 days

issue commentRecordReplay/devtools

Implement an indeterminate loading UI

Don't love it -- we've had that design for ages and I prefer the new ones. Partially because I think they look better, partially because I don't want to re-visit all those hours of animation discussion.

But I care less about the specific animation and more that we refactor the loading screen and get something shipped live.

jaril

comment created time in 2 days

issue commentRecordReplay/devtools

Make print statements discoverable

Thanks @jaril ! This will help me with a task I have open, which is to try and split out some new projects.

jasonLaster

comment created time in 2 days

issue commentRecordReplay/devtools

Animate our loading screen

Ping ping! Friendly reminder that we've been holding off on this since 13 October, and the issue predates that by a few months. We know from design psychology that a loading screen goes a long way towards feeling professional, reliable, and fast.

Also see: https://github.com/RecordReplay/devtools/issues/4541 https://github.com/RecordReplay/devtools/issues/3523

jonbell-lot23

comment created time in 2 days

issue commentRecordReplay/devtools

Implement an indeterminate loading UI

We know from design psychology that a loading screen that looks more professional will a) feel faster b) make people less judgemental when things go wrong. So between this ticket and #4541 this is work we need to address. Ping!

jaril

comment created time in 2 days

issue commentRecordReplay/devtools

Loading screen + privacy tip revisions

Also: we know from design psychology that a loading screen that looks more professional will a) feel faster b) make people less judgemental when things go wrong. So between this ticket and https://github.com/RecordReplay/devtools/issues/3523 this is work we need to address.

jonbell-lot23

comment created time in 2 days

issue commentRecordReplay/devtools

Loading screen + privacy tip revisions

I'm going through old tickets I've filed with design-complete and seeing where we are on them. Ping!

jonbell-lot23

comment created time in 2 days

issue commentRecordReplay/devtools

The upload screen is cropping off the save button

I'm going through old tickets I've filed with design-complete and seeing where we are on them. Ping!

jonbell-lot23

comment created time in 2 days

issue commentRecordReplay/devtools

Console loading state

Ah, great point Jaril

jasonLaster

comment created time in 2 days

issue commentRecordReplay/devtools

Default editor should make search more clear

100% and this ties (or could tie) to the command palette

jonbell-lot23

comment created time in 2 days

push eventRecordReplay/devtools

Jon Bell

commit sha 5cde42c87ee7a18294661f77881c374f34b98f3b

Removing “loading events…” (#4693) * Removing “loading events…” * Remove unused code Co-authored-by: Jaril <jarilvalenciano@gmail.com>

view details

push time in 5 days

PR merged RecordReplay/devtools

Removing “loading events…”

Fixes https://github.com/RecordReplay/devtools/issues/4653

+4 -15

1 comment

1 changed file

jonbell-lot23

pr closed time in 5 days

issue closedRecordReplay/devtools

Drop "Loading events" from Replay Info panel

See https://github.com/RecordReplay/devtools/issues/4653

We're addressing our loading state in our new global loader (the one that's part of our play button) so we don't need this line anymore.

(Hurray for continuing to remove lines of code and features that we don't need!)

closed time in 5 days

jonbell-lot23

issue commentRecordReplay/devtools

Drop "Loading events" from Replay Info panel

Fixed: https://github.com/RecordReplay/devtools/pull/4693#pullrequestreview-817487106

jonbell-lot23

comment created time in 5 days

PullRequestReviewEvent

issue commentRecordReplay/devtools

Explicit click target to add a print statement

Really liking this direction!

jaril

comment created time in 6 days

PR opened RecordReplay/devtools

Removing “loading events…”

Fixes https://github.com/RecordReplay/devtools/issues/4653

+1 -7

0 comment

1 changed file

pr created time in 8 days

create barnchjonbell-lot23/devtools

branch : remove-loading-events-prompt

created branch time in 8 days

issue commentRecordReplay/devtools

Empty state for events panel

Filed a ticket: https://github.com/RecordReplay/devtools/issues/4692

jaril

comment created time in 8 days

issue openedRecordReplay/devtools

Drop "Loading events" from Replay

created time in 8 days

issue closedRecordReplay/devtools

(Placeholder issue for a bunch of stuff I did but got handed off to dev)

image

I did a bunch of stuff this week but many of these items couldn't be counted as "done" because they were handed off to dev. This is a placeholder ticket to refer to that work.

closed time in 8 days

jonbell-lot23

issue commentRecordReplay/devtools

Empty state for events panel

Good question @jasonLaster -- my understanding was that there are multiple stages of things that need to be loaded. If so, you could end up in a state where events have loaded ... but there are other issues that might make the Replay load slowly.

So two questions:

  1. Can this happen? Events loaded but other things slow?
  2. If this can happen, are we ok with our global loader (the circle around the play button) and our per-pane loaders (such as in the diagram above) providing enough loading data? I am, but curious to know your thoughts.

Because if so, yeah! I think we can totally drop it.

jaril

comment created time in 9 days

issue commentRecordReplay/devtools

Outline View should be open by default

On second thought, this is a pure coding task. Removing needs-design and assigning to sprint.

jasonLaster

comment created time in 9 days

issue commentRecordReplay/devtools

Show when there are incomplete results from the backend

I'm going though needs-design tickets so I came across this. Let me know if you'd like help drawing this up in Figma.

jasonLaster

comment created time in 9 days

issue openedRecordReplay/devtools

Default to Replay Info unless there are comments

Description Today we default to the comments pane, which is usually blank. We're going to default to Replay Info ... unless there's a comment.

Scenarios Author of Replay loads Replay for first time: default to Replay info Author of Replay loads Replay every other time: default to last place they were

Someone loads Replay for the first time and there's no comment: default to Replay info Someone loads Replay for the first and there is a comment: default to Comment Someone loads Replay every other time: default to the last place they were

created time in 9 days

issue commentRecordReplay/devtools

Replay Info Pane should include the last URL

Also see https://github.com/RecordReplay/devtools/issues/4407, it's part of the same redesign.

image

Jaril note: I used non-Material design icons here. To discuss.

jasonLaster

comment created time in 9 days

more