Welcome to Codidact Meta!
Codidact Meta is the meta-discussion site for the Codidact community network and the Codidact software. Whether you have bug reports or feature requests, support questions or rule discussions that touch the whole network – this is the site for you.
Activity for Oleg Valterâ€
Type | On... | Excerpt | Status | Date |
---|---|---|---|---|
Comment | Post #294044 |
This is, as far as I am aware, a design limitation of how we implemented comments: when you apply a suggestion, it's inserted into the text area in a specific format (`@#` followed by user id that you've chosen).
Agreed that we can do better here, but it'll require us to use a proper editor engin... (more) |
— | 5 days ago |
Comment | Post #293691 |
Since everyone's is favor (myself included), the line will be removed once (provided there are no objections during review) [1630](https://github.com/codidact/qpixel/pull/1630) is merged in. (more) |
— | 17 days ago |
Comment | Post #293942 |
For the benefit of other readers: as per discussion on the abovementioned GitHub issue, SE editor's behavior doesn't seem to be intentional - will proceed with switching to per-line wrapping. (more) |
— | 22 days ago |
Comment | Post #293942 |
Yeah, this is how editor engines usually work (for example, ProseMirror and TipTap that's based on it). Unfortunately, this is not how Stack Overflow's editor works, so I think we should reach consensus on whether the change is warranted (I personally think it is). Backlogged an [issue](https://githu... (more) |
— | 22 days ago |
Comment | Post #293940 |
Yeah, we certainly can afford to show more buttons at the first level if the viewport width allows, but given that I'd like to avoid making it JS-based, the change will take some time. In the meantime, we'll soon add most of these buttons to the first level anyways (your previous report jumpstarted t... (more) |
— | 22 days ago |
Comment | Post #293924 |
An unfortunate side-effect of us phasing out JQuery, should be [fixed soon](https://github.com/codidact/qpixel/pull/1614) (more) |
— | 25 days ago |
Comment | Post #293258 |
That one is on me - I was impelementing an unrelated fix and haven't noticed that it made an implicit return. To make matters worse, this slipped through the tests (both automated and manual as I was focused on a specific behavior). [Fix done](https://github.com/codidact/qpixel/pull/1508) and pending... (more) |
— | 5 months ago |
Comment | Post #293215 |
Not sure how to reproduce it. There is an explicit call for `do_draft_delete` on post creation. Anything specific about the Q&A pair or the circumstances that you can think of, @#53890? (more) |
— | 5 months ago |
Comment | Post #293213 |
Fixed as per [1506](https://github.com/codidact/qpixel/pull/1506), pending deployment. Implemented your suggestions in full, they all look quite reasonable. (more) |
— | 5 months ago |
Comment | Post #293037 |
There was a weird edge case in that if a post draft gets saved while it has no tags, an empty tag is persisted in the cache. Should be fixed after the [PR addressing the issue](https://github.com/codidact/qpixel/pull/1458) is merged in. (more) |
— | 6 months ago |
Comment | Post #292574 |
This is a known issue, yeah, we are working on resolving this: https://github.com/codidact/qpixel/issues/1353 (more) |
— | 9 months ago |
Comment | Post #292446 |
Yeah, it's been annoying me for a while now too - expect a fix as soon as I have some time to spare. (more) |
— | 9 months ago |
Comment | Post #287698 |
[This PR](https://github.com/codidact/qpixel/pull/1228) should address the issue, as well as add a bunch of missing title attributes to other headers. As is tradition: pending review, merge, and deployment. (more) |
— | over 1 year ago |
Comment | Post #287763 |
[This PR](https://github.com/codidact/qpixel/pull/1226) should add the missing score tooltips, for now awaiting review, merge, and deployment. (more) |
— | over 1 year ago |
Comment | Post #288443 |
[This PR](https://github.com/codidact/qpixel/pull/1224) should make the score updates without reloading the page possible - awaiting review, merge, and deployment. (more) |
— | over 1 year ago |
Comment | Post #288400 |
[This PR](https://github.com/codidact/qpixel/pull/1222) should address the problem, for now pending review, merge, and deployment. (more) |
— | over 1 year ago |
Comment | Post #288508 |
This should be fully addressed by [#1221](https://github.com/codidact/qpixel/pull/1221) pending review, merge, and deployment. (more) |
— | over 1 year ago |