Recent activity

Re: wmenu crashes in sway with output scale > 1 2 days ago

From Umar Getagazov to ~adnano/wmenu-devel

Yes, I can confirm that this patch fixes the issue.

Re: wmenu crashes in sway with output scale > 1 4 days ago

From Umar Getagazov to ~adnano/wmenu-devel

On Mon Jan 30, 2023 at 10:43 PM +03, Adnan Maolood wrote:
> Hmm. I can't reproduce this. What error message are you getting?

$ ./wmenu
wl_surface@3: error 2: Buffer size (1728x19) is not divisible by scale (2)

Do note that my display's resolution is 3456x2160 with scale factor 2.
(So the buffer seems to have the old size, yet the surface is being
committed.)

wmenu crashes in sway with output scale > 1 4 days ago

From Umar Getagazov to ~adnano/wmenu-devel

Happens on Sway 1.8. See this:

https://todo.sr.ht/~exec64/imv/20
https://lists.sr.ht/~exec64/imv-devel/patches/38286

Unfortunately I don't know any Wayland so I can't write a patch, but
removing the line main.c:414 seems to resolve the issue.

Re: [PATCH hire.sr.ht v2] api: expand the schema and implement all methods 9 days ago

From Umar Getagazov to ~sircmpwn/sr.ht-dev

On Mon Jan 23, 2023 at 12:53 PM +03, Drew DeVault wrote:
> On Sat Dec 24, 2022 at 7:56 AM CET, Umar Getagazov wrote:
> > +// UpdateProfile is the resolver for the updateProfile field.
> >  func (r *mutationResolver) UpdateProfile(ctx context.Context, input map[string]interface{}) (*model.Profile, error) {
>
> I don't love how this code path mixes inserting new profiles with
> updating old profiles. Can we split them up?

Do you want the resolver to be split into two (createProfile,
updateProfile), or code inside the resolver (like if the user has a
profile, run a separate update branch, else run a separate creation
branch)?

[PATCH hire.sr.ht] api: update core-go and gqlgen 11 days ago

From Umar Getagazov to ~sircmpwn/sr.ht-dev

---
 api/go.mod                    |  23 ++++---
 api/go.sum                    | 118 +++++++++++++---------------------
 api/gqlgen.yml                |   2 +
 api/graph/schema.resolvers.go |   8 +++
 4 files changed, 67 insertions(+), 84 deletions(-)

diff --git a/api/go.mod b/api/go.mod
index 974d9f5..ce0f2a9 100644
--- a/api/go.mod
+++ b/api/go.mod
@@ -3,10 +3,10 @@ module git.sr.ht/~sircmpwn/hire.sr.ht/api
go 1.17

[message trimmed]

[PATCH hire.sr.ht] schema.sql: fix table definitions 11 days ago

From Umar Getagazov to ~sircmpwn/sr.ht-dev

Change `serial` to `PRIMARY KEY GENERATED BY DEFAULT AS IDENTITY` which
is the SQL-standard way of defining an auto-incrementing PK, and while
we're at it, fix my `DEFAULT` typo introduced in the commit
733c3fe019ff2accdaf7f2e3673d0e68d86adc39.
---
 schema.sql | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/schema.sql b/schema.sql
index 3b7aacf..ebdb032 100644
--- a/schema.sql
+++ b/schema.sql
@@ -57,9 +57,9 @@ CREATE TYPE unit AS ENUM (
);
[message trimmed]

Re: [PATCH hare] cmd/haredoc: fix submodule references a month ago

From Umar Getagazov to ~sircmpwn/hare-dev

On Fri Dec 23, 2022 at 3:33 PM +03, Drew DeVault wrote:
> This causes the link for crypto::aes to lead to /crypto/crypto/aes when
> tested in Firefox with python3 -m http.server. *Test* your changes
> before you send the patch, please.

I noticed this conversation and thought I should chime in yet again.

To ensure we're all on the same page, this is how relative URLs work[0]:

	/crypto + crypto/aes   = /crypto/aes
	/crypto/ + crypto/aes  = /crypto/crypto/aes

Links generated by haredoc (which point to directories) don't have
trailing slashes. HTTP servers behave differently in regards to how

[PATCH hire.sr.ht v2] api: expand the schema and implement all methods a month ago

From Umar Getagazov to ~sircmpwn/sr.ht-dev

---
Not sure why the GraphQL schema has all documentation converted to
comments. I stuck to the already establihed convention in this project,
but I wonder if this was a mistake or a deliberate decision?

Changes since v1:
- Run cleanup() after failing to retry a transaction
- Sleep for a short amount of time before retrying a transaction

 api/go.mod                    |  65 ++--
 api/go.sum                    | 517 +++++----------------------
 api/gqlgen.yml                |   2 +
 api/graph/model/pricing.go    |  10 +
 api/graph/model/profile.go    |  21 ++
[message trimmed]

[PATCH hire.sr.ht] api: expand the schema and implement all methods a month ago

From Umar Getagazov to ~sircmpwn/sr.ht-dev

---
Not sure why the GraphQL schema has all documentation converted to
comments. I stuck to the already establihed convention in this project,
but I wonder if this was a mistake or a deliberate decision?

 api/go.mod                    |  65 ++--
 api/go.sum                    | 517 ++++++---------------------
 api/gqlgen.yml                |   2 +
 api/graph/model/pricing.go    |  10 +
 api/graph/model/profile.go    |  21 ++
 api/graph/model/user.go       |  10 +-
 api/graph/resolver.go         |   6 +
 api/graph/schema.graphqls     |  30 +-
 api/graph/schema.resolvers.go | 651 ++++++++++++++++++++++++++++++++--
[message trimmed]

Re: [PATCH mycorrhiza 0/1] Draft center search bar with standard flexbox 2 months ago

From Umar Getagazov to ~bouncepaw/mycorrhiza-devel

Here's what could be a correct version of this patch, by the way.

diff --git a/static/default.css b/static/default.css
index 8d09d53..ee946cc 100644
--- a/static/default.css
+++ b/static/default.css
@@ -686,10 +686,13 @@ a.btn_destructive:visited,

.top-bar__search-bar {
       border: 0;
       vertical-align: -webkit-baseline-middle; /* You have no idea how sorry I am. This is a non-standard value. Seems to work in Safari, Opera and mobile Chrome. We need a jedi and a knight in shiny armor to replace it with something more standard. */
       width: 100%;
}
[message trimmed]