June 11, 2016 at 5:00 am #3694Tobias BinnaGuest
First of all, thanks for the great Book Review plugin. This saved me a lot of time to develop on my own.
Now, when I wanted to configure the plugin for a client I noticed a few small things that I wanted to place as suggestions to improve:
book-review-public.phpit would be good if the top div has a class attribute
book-reviewinstead of a id attribute
<div class="book-review" ...
Semantically id here is not correct because I potentially can have multiple divs with the same id on my page.
class-book-review-public.phpit would be good to only add
$style = 'border-style: solid;';if
$border_widthis actually set or greater then 0. My problem here is that I cannot remove the border. If
border-styleis added conditionally, I can choose not to render the border.
I sometimes paste ISBN numbers from online resources which include dashes (-). Would be nice to do a simple
str_replaceto remove all dashes from the input string before calling the API. This would save me the little work of removing them every time before I submit the request.
Feel free to contact me if you have any questions on my suggestions.
Thanks!June 12, 2016 at 9:29 am #3704
Thank you for your suggestions!
Can you provide a use case where there could possibly be more than one book review on a single page or post? I’m wary of changing the ID to a class because others may have already written CSS that attaches to #book-review, so I’d like to clarify the use case before deciding what to do.
Would you care to submit a pull request on Github to address your other suggestions?
Thx.June 13, 2016 at 11:46 am #3705Tobias BinnaGuest
I think on a single page or post it won’t happen that you have two reviews. I was more thinking of a blog feed where each feed entry is a post with a book review. I think it is ok, it works and I understand that your concern of breaking other peoples code or styles.
No problem, I will submit a pull request for the other issues. Thanks for the link to the repo.June 29, 2016 at 6:28 am #3737
Thanks for the pull request. If all is well, it will go out with the next release, although I’m not sure when that will be just yet.August 28, 2016 at 2:26 pm #3823
Your changes went out in version 2.3.8 of the plugin that was released yesterday. Thanks so much for the PR!
- The forum ‘Feature Requests’ is closed to new topics and replies.