Code review comment for lp:~gary-lasker/software-center/tos_buy_button_fix_lp961216

Revision history for this message
Gary Lasker (gary-lasker) wrote :

Yes, as I said in my initial comment, this MP is for the details page case. I have the code in progress to fix the list view case, and that would have followed today with a second MP.

This one is a very small change as I was trying to make a "surgical" fix with minimal code churn at this late date in the cycle, and in fact the fix does the trick fine with no regressions. But if there is an alternate and cleaner approach that will take care of both the listview and detailsview cases, I'll find that now and implement it.

Thanks mvo!

« Back to merge proposal