pagure

#3381 fix pr-dropdown messed up label for forks
Merged by pingou. Opened by ryanlerch.
ryanlerch/pagure pr-dropdown  into  master

Download 3381.patch

Fixes the weird rendering of the labels in the
New PR dropdown when listing branches from Forks

Fixes #3376

This also styles up the labels to be the same as
in the PR page.

Screenshot_from_2018-06-28_12-17-55.png

left over debugging?

This seems a little odd, was it changed by accident before?

This looks a little odd, if you're in a fork this won't work no?

@pingou the diff looks a little weird because i swapped the order of the forks and main branches ajax calls, so it kinda looks like they are wrong looking at the diffs, but its because the code is similar, but slightly different...

rebased onto e423f0b5737c060231366ebb162c024d31c1c9d9

We could drop the g.authenticated since we're checking this just above, I'll take care of this

rebased onto 86837c0623220845b0ceec271c20a00f52b46ccc

Local tests looks fine, let's re-run jenkins and merge :)

Pretty please pagure-ci rebuild

Pull-Request has been merged by pingou
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.