Rajasthan, India
Account created on 30 September 2016, over 7 years ago
  • Software Development Engineer III at Valueboundย  โ€ฆ
#

Merge Requests

Recent comments

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

The field label inline issue is resolved. Thanks for reviewing it.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

anoopsingh92 โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

The Issue does not exist, It's working fine with the attribute CSS selector.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India
๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

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.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

Thank you so much @all for the contribution.
Merged, Now pager and mini pager is looking good.

Thank you

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

The code is updated and the branch has also changed.
Please review it now. Thanks.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

Thanks for the contribution @Shubham_Kumar.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

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.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

Thank you @shubham_kumar for identifying that issue.
That issue is fixed in the dev release.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

anoopsingh92 โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

Issues do not exist. Please review it.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

That issue was resolved in the latest version.
Thanks for the contribution.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

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

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

Okay @ptmkenny, Understood.
Thank you.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

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

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

Thanks for the verification @yashaswi18.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

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

Video โ†’

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

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.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

Drupal coding standard fixes are done.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

anoopsingh92 โ†’ created an issue.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

This issue is resolved in the latest version. Please review.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

Committed the PR.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India
๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India
๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

Please review.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

anoopsingh92 โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia anoopsingh92 Rajasthan, India

I have done @Anybody, Please review the Merge request !6
Thanks

Production build 0.69.0 2024