THEME: Lyn Portfolio – 1.0.4

[mienle]

Lyn Portfolio - 1.0.2

Lyn Portfolio theme is designed for personal portfolio with stunning design to catch the attension of your visitors. If you are a freelancer, designers, developers, photographers artists and all other professionals and want to build your Portfolio website then Lyn Portfolio is a perfect theme for you. There is no doubt that, Lyn Portfolio will make your Website look more impressive and attractive. Lyn Portfolio is a very easy-to-use, easily customizable WordPress theme that helps you to build your website in a few minutes. This theme comes with a retina-ready display and responsive layout. It is made mobile-friendly and cross-browser compatible as well so that mobile users can easily access it through any popular web browser. Check the theme documentation and detail at: ​https://crthemes.com/downloads/lyn-portfolio-wordpress-themes/

Theme URL - ​https://crthemes.com/downloads/lyn-portfolio-wordpress-themes/
Author URL - ​https://crthemes.com/

Trac Browser - https://themes.trac.wordpress.org/browser/lyn-portfolio/1.0.2
WordPress.org - ​https://wordpress.org/themes/lyn-portfolio/

SVN - ​https://themes.svn.wordpress.org/lyn-portfolio/1.0.2
ZIP - ​https://downloads.wordpress.org/theme/lyn-portfolio.1.0.2.zip?nostats=1
Live preview – ​https://playground.wordpress.net/#…

Diff with previous version: [234418] https://themes.trac.wordpress.org/changeset?old_path=lyn-portfolio/1.0.1&new_path=lyn-portfolio/1.0.2

History:

Ticket	Summary	Status	Resolution	Owner
#178784	THEME: Lyn Portfolio – 1.0.0	closed	not-approved	vowelweb
#180165	THEME: Lyn Portfolio – 1.0.1	closed	not-approved	fahimmurshed
#181485	THEME: Lyn Portfolio – 1.0.4	closed	not-approved	kafleg
(this ticket)
#183242	THEME: Lyn Portfolio – 1.0.6	reviewing		acosmin

Theme Check Results:

• RECOMMENDED: No reference to add_editor_style() was found in the theme. It is recommended that the theme implement editor styling, so as to make the editor content match the resulting post output in the theme, for a better user experience.

[mienle]

I fixed issues.

• Fix long title issue ​https://prnt.sc/4L91AC5GACBw
• Fix the buttons color issue ​https://prnt.sc/Fuj87-N7K6e-
• Fix image alignment ​https://prnt.sc/a-0vGP7_Biz1
• No submenu icon!
• Have to fix the keyboard navigation!

Please help me review this theme. Thank you.

[kafleg]

Hello @mienle

There are still some issues on your theme. Please check it.

Site title and Tagline

• Default logo is not allowed. The site title and tagline should be displayed.
• The site title and tagline options is not working. ​https://i.rankmath.com/i/IXMFBf

Prefixing

Here’s a list of the most common things that should be prefixed:

• PHP function names.
• PHP class names.
• PHP global variables.
• Action/Filter hooks.
• Script handles.
• Style handles.
• Image size names.

Read more here: ​https://themereview.co/prefix-all-the-things/

• Invalid functions name prefixing, customizer_repeater_sanitize, sanitize_sections_order etc. Check sanitize_callback.php.

Minified and Compressed Files

• minified and compressed files are not allowed to bundled in theme unless you've original files bundled. If you have minified version of JS,CSS it's original files must be in the theme folder.

Development Files =

• Development files should be removed and you can host those development files in GitHub or similar other repository/.

Take reference from already live theme, review their codes and file structure and fix the issues on your theme accordingly.

Regards
KafleG

[themetracbot]

Lyn Portfolio - 1.0.3

Lyn Portfolio theme is designed for personal portfolio with stunning design to catch the attension of your visitors. If you are a freelancer, designers, developers, photographers artists and all other professionals and want to build your Portfolio website then Lyn Portfolio is a perfect theme for you. There is no doubt that, Lyn Portfolio will make your Website look more impressive and attractive. Lyn Portfolio is a very easy-to-use, easily customizable WordPress theme that helps you to build your website in a few minutes. This theme comes with a retina-ready display and responsive layout. It is made mobile-friendly and cross-browser compatible as well so that mobile users can easily access it through any popular web browser. Check the theme documentation and detail at: ​https://crthemes.com/downloads/lyn-portfolio-wordpress-themes/

Theme URL - ​https://crthemes.com/downloads/lyn-portfolio-wordpress-themes/
Author URL - ​https://crthemes.com/

Trac Browser - https://themes.trac.wordpress.org/browser/lyn-portfolio/1.0.3
WordPress.org - ​https://wordpress.org/themes/lyn-portfolio/

SVN - ​https://themes.svn.wordpress.org/lyn-portfolio/1.0.3
ZIP - ​https://downloads.wordpress.org/theme/lyn-portfolio.1.0.3.zip?nostats=1
Live preview – ​https://playground.wordpress.net/#…

Diff with previous version: [234889] https://themes.trac.wordpress.org/changeset?old_path=lyn-portfolio/1.0.2&new_path=lyn-portfolio/1.0.3

History:

Ticket	Summary	Status	Resolution	Owner
#178784	THEME: Lyn Portfolio – 1.0.0	closed	not-approved	vowelweb
#180165	THEME: Lyn Portfolio – 1.0.1	closed	not-approved	fahimmurshed
#181485	THEME: Lyn Portfolio – 1.0.4	closed	not-approved	kafleg
(this ticket)
#183242	THEME: Lyn Portfolio – 1.0.6	reviewing		acosmin

Theme Check Results:

• RECOMMENDED: No reference to add_editor_style() was found in the theme. It is recommended that the theme implement editor styling, so as to make the editor content match the resulting post output in the theme, for a better user experience.

[kafleg]

Hello @mienle

• The top header section is hidden, ​https://i.rankmath.com/i/l5wqVw
• Default logo please remove from footer, ​https://i.rankmath.com/i/v64SAj
• The license of image mentioned in the readme file is, ​https://pxhere.com/en/photo/656755 and the actual image in the screenshot is different.

Please check your theme properly, license and design.

Thank you
KafleG

[themetracbot]

Lyn Portfolio - 1.0.4

Lyn Portfolio Resume is multipurpose portfolio and resume theme for personal portfolio and resume with stunning design to catch the attension of your visitors. If you are a freelancer, designers, developers, photographers artists and all other professionals and want to build your Portfolio website then Lyn Portfolio is a perfect theme for you. There is no doubt that, Lyn Portfolio will make your Website look more impressive and attractive. Lyn Portfolio is a very easy-to-use, easily customizable WordPress theme that helps you to build your website in a few minutes. This theme comes with a retina-ready display and responsive layout. It is made mobile-friendly and cross-browser compatible as well so that mobile users can easily access it through any popular web browser. Check the theme documentation and detail at: ​https://crthemes.com/downloads/lyn-portfolio-wordpress-themes/

Theme URL - ​https://crthemes.com/downloads/lyn-portfolio-wordpress-themes/
Author URL - ​https://crthemes.com/

Trac Browser - https://themes.trac.wordpress.org/browser/lyn-portfolio/1.0.4
WordPress.org - ​https://wordpress.org/themes/lyn-portfolio/

SVN - ​https://themes.svn.wordpress.org/lyn-portfolio/1.0.4
ZIP - ​https://downloads.wordpress.org/theme/lyn-portfolio.1.0.4.zip?nostats=1
Live preview – ​https://playground.wordpress.net/#…

Diff with previous version: [235332] https://themes.trac.wordpress.org/changeset?old_path=lyn-portfolio/1.0.3&new_path=lyn-portfolio/1.0.4

History:

Ticket	Summary	Status	Resolution	Owner
#178784	THEME: Lyn Portfolio – 1.0.0	closed	not-approved	vowelweb
#180165	THEME: Lyn Portfolio – 1.0.1	closed	not-approved	fahimmurshed
#181485	THEME: Lyn Portfolio – 1.0.4	closed	not-approved	kafleg
(this ticket)
#183242	THEME: Lyn Portfolio – 1.0.6	reviewing		acosmin

Theme Check Results:

• RECOMMENDED: No reference to add_editor_style() was found in the theme. It is recommended that the theme implement editor styling, so as to make the editor content match the resulting post output in the theme, for a better user experience.
• REQUIRED: Found a reference to <code>unsplash.com</code>. Assets from this website does not use a license that is compatible with GPL. <a href="​https://unsplash.com/license" target="_blank">View license (opens in a new window).</a>

[mienle]

Hi @kafleg

I fixed issues.

Thank you so munch.

[kafleg]

Hi!
I have had a quick look at the theme and at this point, it does not follow the requirements.

The Theme Review Team does not hard reject themes, -we want you to resubmit your theme, but we need you to read the requirements more carefully.

If the requirements are unclear, let us know so that we can continue to improve them.

When a ticket is closed as not approved, it loses its position in the review queue.

When the update is submitted, a new ticket is created at the end of the queue.
​https://make.wordpress.org/themes/handbook/review/required/
​https://make.wordpress.org/themes/handbook/review/how-to-do-a-review-draft/

Use this plugin to check your theme,
​https://wordpress.org/plugins/theme-check/

Use this upload form to submit your update: ​https://wordpress.org/themes/upload/

Example Issues

• Did you really check the keyboard navigation by adding sub-menus? I'm not able to access the sub-menus using keyboard navigation.

• Sticky header hides the latest post title, ​https://i.rankmath.com/i/tkU33h
• There are many text areas in your theme customizer. Content creation is not allowed.
• Default value in the customizer is loading locally, ​https://prnt.sc/5n_Jb6-1bZhH
• Please check the customizer setting properly, ​https://prnt.sc/xEG6wJlsrCHV

I don't think you check it properly. Your theme won't get approved in this current state.

Thank you
KafleG

[mienle]

I fixed issues,

1. Fixed ​https://prnt.sc/N1VUzJ-pVCBr
2. Fixed ​https://prnt.sc/T9-cuJjXgDpV

3,4 Fixed

Thanks you, Good day.