The field label inline issue is resolved. Thanks for reviewing it.
anoopsingh92 โ made their first commit to this issueโs fork.
The Issue does not exist, It's working fine with the attribute CSS selector.
Thanks @ravi-kant for the quick response.
@ugintl you can follow the steps.
- Create a Sub Theme, Go through the Readme.md file for the Sub-theme creation steps.
- Go to the FlexiStyle Bootstrap theme, copy the components folder and page.html.twig from the templates folder, and paste it into your custom theme templates folder.
- If you want to customize the Front page, clone the page.html.twig into the page--front.html.twig.
- If you have minor CSS changes then you can add them directly into the style.css file.
- If you want to add a separate CSS file for the front-page, Then you can add a new library in your themes libraries file.
Create a Library in your custom theme libraries file such as.
# Front Page Library
front_page:
css:
component:
css/front.css: { }
Attaching a library via a Twig template
You can attach an asset library to a Twig template using the attach_library()
function in your page--front.html.twig.
{{ attach_library('custom_theme/front_page') }}
Attach the Library in the Preprocess Function for the Front Page
use Drupal\Core\Url;
/**
* Implements hook_preprocess_HOOK() for page templates.
*/
function custom_theme_preprocess_page(array &$variables) {
// Check if the current page is the front page.
if (\Drupal::service('path.matcher')->isFrontPage()) {
$variables['#attached']['library'][] = 'custom_theme/front_page';
}
}
Note: "custom_theme" replace it with your custom theme name.
Thank you so much @all for the contribution.
Merged, Now pager and mini pager is looking good.
Thank you
The code is updated and the branch has also changed.
Please review it now. Thanks.
Thanks for the contribution @Shubham_Kumar.
You can't remove that component library from the flexi_style.libraries.yml file.
Need to create a twig for the mini page it will be resolved. instead of doing any changes in CSS file.
Thank you @shubham_kumar for identifying that issue.
That issue is fixed in the dev release.
anoopsingh92 โ made their first commit to this issueโs fork.
Issues do not exist. Please review it.
That issue was resolved in the latest version.
Thanks for the contribution.
Thanks for the feedback @apaderno,
I worked on your mentioned comment
#10 โ
.
I have updated the file flexi_style.theme please review it again.
Thank you
Okay @ptmkenny, Understood.
Thank you.
Hi @vishal.kadam,
First of all, thank you so much for reviewing that project.
I understand your concern. I want to let you know, that I am not creating multiple project applications for getting that role, that was not my intention Vishal.
My First application (FlexiStyle) has been completed. This is a pure CSS based Drupal theme. We are not using any third-party CSS library like Bootstrap, MDL, or Foundation, etc. in this project. this is a base theme for other projects. we can create a sub-theme based on it.
My Second application (FlexiStyle Bootstrap) is under development. You can check the status of the application. This application is using third-party CSS library Bootstrap. You can review the code as well. This project is different from my first project (FlexiStyle).
Both projects are different.
When I created a ticket for security advisory coverage, mistakenly i sent it to review. Now I can change the ticket status to Active. I am tactically working on it. when it will be completed then i will send it to review.
If you need any kind of information please let me know.
Thank you
anoopsingh92 โ created an issue.
Thanks for the verification @yashaswi18.
Hi @Hamid.ali,
Updating the branch. That issue does not exist in the latest deployment. please review it again.
Testing environment on simplytest.me
URL: https://master-nurd9vgysl7of2ieulmjm5twbs2pizkl.tugboatqa.com/
Username/password: admin/admin
Thanks for the review @vishal.kadam, I think, I missed it for the CSS files.
Now, I have fixed all the mentioned issues, you can check and review them.
Thanks.
Drupal coding standard fixes are done.
anoopsingh92 โ created an issue.
This issue is resolved in the latest version. Please review.
Committed the PR.
Please review.
anoopsingh92 โ made their first commit to this issueโs fork.
I have done @Anybody, Please review the Merge request !6
Thanks