- Group Reviewers
- R120:4dca72bdf59a: Kicker/RecentDocuments: add icons to actions
I would have needed to edit Kicker::createActionItem, it might be interesting.
But not keeping the const is not too important those variables have a limited lifetime anyway.
Not having const here does not add a copy or anything.
I am not too fond of this version growing the number of parameters of Kicker::createActionItem, and given its third parameter before icon is QVariant this becomes error-prone.
There are a lot more usage of this method that I haven't yet updated, if we want to follow this course.