Adam Setch comments

Results 347 comments of


                                            Adam Setch

feat: zoom slider component

> Should be good now @setchy Looks great @afonsojramos, thank you! What's your thoughts on if the slider should reflect the manually set zoom level via `Cmd + -` or...

feat: zoom slider component

> > Also, the setting doesn't seem to be preserved across restarts..? > > @setchy @adufr I've been trying to set that up (even with the settings, with no success),...

feat: zoom slider component

You made me think? Should we only allow zoom out?

feat: zoom slider component

I noticed some odd slider behavior under the following conditions: - click + hold the slider and move left or right. seems to glitch a fair bit for me. -...

feat: zoom slider component

Appreciate the time spent @afonsojramos. 🙏 I hope this is user error on my part, but, I cannot get the zoom settings to load on app restart... Sometimes it works,...

feat: zoom slider component

> > Alternatively, what if we: > > > > * had a Zoom label in Appearance Settings > > * it had a simple zoom in, zoom out, reset...

Move off of `nodeIntegration` to `contextIsolation:true`

See https://www.electronjs.org/docs/latest/api/context-bridge

fix(auth): Support 2FA via browser window auth

superseded by https://github.com/gitify-app/gitify/pull/1781

Sanitize externalReferences URLs and exclude invalid ones + Strict JSON Schema validation

Great work @marob Tested this branch against some of my repos. BOM looks good. Uploaded to DTrack 4.11.1 with BOM Validation enabled without issues One observation: we may need to...

Filter out commit notifications

Thanks for being a Gitify user @msjtheriault. In the above scenario, what do these notifications look like within the GitHub Web UI - https://github.com/notifications I was thinking that our `Reason`...