helloflask / bootstrap-flask

Bootstrap 4 & 5 helper for your Flask projects.
https://bootstrap-flask.readthedocs.io
Other
1.1k stars 189 forks source link

Add an optional parameter for render_table() #287

Open grant-sea opened 1 year ago

grant-sea commented 1 year ago

Add an optional parameter for render_table() to allow setting the location of the actions column. The two options are "front" (the 1st column of the table) and "end" (default value, the last column of the table).

PanderMusubi commented 1 year ago

Looks nice. @greyli how about actions_front=True using a bool instead of a string?

grant-sea commented 1 year ago

Looks nice. @greyli how about actions_front=True using a bool instead of a string?

Yes. that's also my first thought. I was struggling about that. But you are right, a bool will be better. And I found a more compact way to implement this function after I read Greyli's Flask book :). How can I recall this PR and edit it again? Thanks for the hlep.

PanderMusubi commented 1 year ago

put your new edits in a new commit with a new commit message, then run git rebase -i HEAD~2

change the first word before the commit message of the latest commit into squash

save and exit that file, then remove the new commit messages and leave only the old commit message at the top of the next file.

save and exit that and do git push -f which will push it here and override the old commit.

grant-sea commented 1 year ago

put your new edits in a new commit with a new commit message, then run git rebase -i HEAD~2

change the first word before the commit message of the latest commit into squash

save and exit that file, then remove the new commit messages and leave only the old commit message at the top of the next file.

save and exit that and do git push -f which will push it here and override the old commit.

Thanks for the instructions. But it seems quite complicated to me. I only use the github website operations. I suppose I have done the new commit. But I don't know how to move the PR from Draft back to acitve. Could you please help? Thanks!

PanderMusubi commented 1 year ago

Just make another commmit and push that. @greyli can merge them later.

grant-sea commented 1 year ago

Just make another commmit and push that. @greyli can merge them later.

I have moved the PR back to open status (2nd commit done). Thanks!