Milan Cvetkovic

Results 41 comments of Milan Cvetkovic

@tordans > I wonder if we can get rid of the special font size on the intro paragraph "Free and editable…" and make it the same size as the "by...

@tordans Thanks for the review. Updated spacing on login screen above horizontal line. > * The new forgot-passwort-field looks non-ideal on mobile Fixed based on your advice. > **Column spacing...

@tordans > Just double checking: We cannot trigger the UI that is used for the app-permissions-flow on that server, right? (Just showing it, I don't want to submit…) You can...

@tomhughes, are there any more modifications required for this PR?

> I'll try and have a proper look again at the weekend but one thing I did just notice is that the login screen still has a trailing colon on...

I think it would be useful to be able select changeset comments relevant to a user - which would include comments to all changesets they are subscribed to. Something along...

> That's a potentially very expensive query though so I'm not sure we want to be doing that in the live API. I understand the reluctance, one has to be...

It shouldn't be too bad. It is the number of _comments_ on their subscribed changesets that is the limiting factor, rather than number of subscribed changesets. Inner join should get...

> The current record appears to be a user that is subscribed to nearly 650000 changesets ;-) Wow! Probably the number of changeset comments for him/her is also in thounsands

> > I think it would be useful to be able select changeset comments relevant to a user - which would include comments to all changesets they are subscribed to....