Home › Support › Feature Requests › Improvments
- This topic has 4 replies, 2 voices, and was last updated 5 years, 10 months ago by
Donna.
-
AuthorPosts
-
June 11, 2016 at 5:00 am #3694
Tobias Binna
GuestFirst 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:
-
In
book-review-public.php
it would be good if the top div has a class attributebook-review
instead of a id attributebook-review
:<div class="book-review" ...
Semantically id here is not correct because I potentially can have multiple divs with the same id on my page. -
In
class-book-review-public.php
it would be good to only add$style = 'border-style: solid;';
if$border_width
is actually set or greater then 0. My problem here is that I cannot remove the border. Ifborder-style
is 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_replace
to 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 #3704Donna
KeymasterHi Tobias,
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 Binna
GuestHey Donna,
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 #3737Donna
KeymasterHi Tobias,
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 #3823Donna
KeymasterHi Tobias,
Your changes went out in version 2.3.8 of the plugin that was released yesterday. Thanks so much for the PR!
-
In
-
AuthorPosts
- The forum ‘Feature Requests’ is closed to new topics and replies.