Easy News is a theme that implements Uikit libraries and stylling. Uikit contains many features that makes theming easier. It also provides a straightforward and structured implementation of for example responsive design, navigation and animations. It also comes with more advanced components such as accordions, parallax and slideshows. All this can be easily introduced straight into Drupal`s twig templates.

This subtheme serves two purposes. Firstly, it aims at a structured but responsive and modern design for news publishing sites. Secondly, it is also an attempt to bring module deployment into the twig templates, the same way as Uikit can be handled. Some examples of this are mentioned in the project's documentation.

Link to project: https://www.drupal.org/project/easy_news

Git clone command: git clone --branch 8.x-1.x https://git.drupal.org/project/easy_news.git easy_news

Demo site: http://vivirjapon.es/

CommentFileSizeAuthor
#18 easy-news-comment.jpg59.38 KBlias
#18 easy-news-user-acct.jpg84.1 KBlias
#18 easy-news-hmpg.jpg81.63 KBlias
theme-snapshot.png113.42 KBMikael Berger
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Mikael Berger created an issue. See original summary.

Mikael Berger’s picture

Mikael Berger CreditAttribution: Mikael Berger as a volunteer commented
Issue summary: View changes
PA robot’s picture

PA robot CreditAttribution: PA robot commented
Issue summary: View changes

Fixed the git clone URL in the issue summary for non-maintainer users.

We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)

Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).

I'm a robot and this is an automated message from Project Applications Scraper.

deepanker_bhalla’s picture

deepanker_bhalla CreditAttribution: deepanker_bhalla as a volunteer and at Srijan | A Material+ Company commented
Status: Needs review » Needs work

Hi Mikael Berger,

As per the automated review of your project:

1) Your Readme.txt is not present as it is needed to be there to meet the guidelines for in-project documentation.

2) Remove License.txt as it will be added by drupal.org packaging automatically.


FILE: /root/repos/pareviewsh/pareview_temp/easy_news.info.yml
--------------------------------------------------------------------------
FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES
--------------------------------------------------------------------------
  1 | WARNING | [ ] Remove "version" from the info file, it will be added
    |         |     by drupal.org packaging automatically
 34 | ERROR   | [x] Expected 1 newline at end of file; 0 found
--------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------

Time: 107ms; Memory: 6Mb
  • Codespell has found some spelling errors in your code.
    ./logo.svg:298: OT  ==> TO

    • Can also check for further more information https://pareview.sh/node/1275

    Mikael Berger’s picture

    Mikael Berger CreditAttribution: Mikael Berger as a volunteer commented

    Thank you for your swift review. I have corrected the errors , except the false spelling alert for the binary part of logo.svg: (https://pareview.sh/node/1279)

    Cheers
    Mikael

    Mikael Berger’s picture

    Mikael Berger CreditAttribution: Mikael Berger as a volunteer commented
    Status: Needs work » Needs review
    deepanker_bhalla’s picture

    deepanker_bhalla CreditAttribution: deepanker_bhalla as a volunteer and at HyTechPro.com commented
    Status: Needs review » Needs work

    Hi Mikael Berger,

    I installed the dependent Uikitty latest 8.x-2.1 and afterward tried to install the easy news but it is giving me some error and my error log is:

    Drupal\Core\Config\UnmetDependenciesException: Configuration objects (field.field.node.article.field_easy_news_section) provided by easy_news have unmet dependencies in Drupal\Core\Config\UnmetDependenciesException::create() (line 84 of /var/www/html/sports-web/core/lib/Drupal/Core/Config/UnmetDependenciesException.php).

    Mikael Berger’s picture

    Mikael Berger CreditAttribution: Mikael Berger as a volunteer commented

    Sorry about that, it is because of missing field storage definition. I am out of town right now, but I will provide the file as soon as I can
    .

    Mikael Berger’s picture

    Mikael Berger CreditAttribution: Mikael Berger as a volunteer commented

    File field.storage.node.field_easy_news_section.yml added to project.

    Mikael Berger’s picture

    Mikael Berger CreditAttribution: Mikael Berger as a volunteer commented

    Sorry about the delay. File field.storage.node.field_easy_news_section.yml has been added to project.

    Mikael Berger’s picture

    Mikael Berger CreditAttribution: Mikael Berger as a volunteer commented
    Status: Needs work » Needs review
    Mikael Berger’s picture

    Mikael Berger CreditAttribution: Mikael Berger as a volunteer commented
    Issue summary: View changes
    PA robot’s picture

    PA robot CreditAttribution: PA robot commented
    Status: Needs review » Needs work

    Git clone failed for https://git.drupal.org/project/easy_news.git while invoking http://pareview.sh/pareview/httpsgitdrupalorgprojecteasy_newsgit

    There is a git tag that has the same name as the branch 8.x-1.x. Make sure to remove this tag to avoid confusion.

    I'm a robot and this is an automated message from Project Applications Scraper.

    Mikael Berger’s picture

    Mikael Berger CreditAttribution: Mikael Berger as a volunteer commented
    Status: Needs work » Needs review

    Tag removed

    deepanker_bhalla’s picture

    deepanker_bhalla CreditAttribution: deepanker_bhalla as a volunteer and at Srijan | A Material+ Company commented
    Status: Needs review » Reviewed & tested by the community

    Hi Mikael,

    Sorry for late reply was busy with my project. Thus checked your project and working fine now.

    Will update if found any

    Mikael Berger’s picture

    Mikael Berger CreditAttribution: Mikael Berger as a volunteer commented
    Priority: Normal » Major
    Issue summary: View changes

    The theme has been modified to no longer depend on the (nowadays unsupported) Uikitty base theme.

    Mikael Berger’s picture

    Mikael Berger CreditAttribution: Mikael Berger as a volunteer commented
    Priority: Major » Critical
    lias’s picture

    lias CreditAttribution: lias commented
    FileSize
    easy-news-hmpg.jpg81.63 KB
    easy-news-user-acct.jpg84.1 KB
    easy-news-comment.jpg59.38 KB

    Just installed theme version 8.x-2.2 on fresh Drupal 8.4.2 as per the Getting Started with Demo instructions.

    Did not use any of the additional modules or theme tweaks and was able to reproduce the base look.
    easy news homepage layout

    No errors reported in logs.

    Issues:

    The user account page displayed the sample background below the default user picture. Not having used the background option before not sure what the actual result should be.

    display of user account

    Easy News Comment field displayed on the default view of the article but once the comment was submitted it didn't display beneath the main content.

    easy news comment field

    README.txt file still lists Uikit as requirement.

    easy_news.info.yml also lists easy_news/uikit directory in libraries but it is not included in this version

    Mikael Berger’s picture

    Mikael Berger CreditAttribution: Mikael Berger as a volunteer commented

    Thanks for the review Isabug. I will release a corrected version in short.

    A couple of notes:

    • Yes, I am afraid something has gone wrong with the user profile. There is a mismatch between field names in twig template and demo install files. The fix is easy but might cause problems for some sites
    • I have not been able to reproduce the issue with comments. Could you please doublecheck that you made the new field visible in "Full content" view mode, not "Default"?
    apaderno’s picture

    apaderno
    he/him
    Primary language Italian
    Location Brescia, 🇮🇹 🇪🇺
    CreditAttribution: apaderno as a volunteer commented
    Assigned: Unassigned » apaderno
    Priority: Critical » Normal

    Thank you for your contribution!
    I am going to update your account so you can opt into security advisory coverage now.
    These are some recommended readings to help with excellent maintainership:

    You can find more contributors chatting on the IRC #drupal-contribute channel. So, come hang out and stay involved.
    Thank you, also, for your patience with the review process.
    Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.

    I thank all the dedicated reviewers as well.

    apaderno’s picture

    apaderno
    he/him
    Primary language Italian
    Location Brescia, 🇮🇹 🇪🇺
    CreditAttribution: apaderno as a volunteer commented
    apaderno’s picture

    apaderno
    he/him
    Primary language Italian
    Location Brescia, 🇮🇹 🇪🇺
    CreditAttribution: apaderno as a volunteer commented
    Status: Reviewed & tested by the community » Fixed

    Status: Fixed » Closed (fixed)

    Automatically closed - issue fixed for 2 weeks with no activity.